Skip to content

Fix anagram finder - #24

Closed
dimaush wants to merge 5 commits into
hexlet-codebattle:mainfrom
dimaush:fix-anagram_finder
Closed

dimaush wants to merge 5 commits into
hexlet-codebattle:mainfrom
dimaush:fix-anagram_finder

Conversation

@dimaush

@dimaush dimaush commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

Изменил логику решения: дубликаты теперь не считаются (кажется, это более логично, чем считаются).

Также внёс разные стилистические правки.

@vtm9

vtm9 commented Oct 16, 2026

Copy link
Copy Markdown
Contributor

Closing after review: the existing task explicitly allows duplicate words, and its solution and asserts consistently implement that contract. This PR intentionally changes that rule to deduplication, changing valid answers such as ["a", "a"] from [["a", "a"]] to the no-anagrams sentinel. There is no demonstrated defect in the original behavior, so this is an unsupported task-rule change rather than a fix. Formatting and naming cleanups can be proposed separately while preserving the contract.

@vtm9 vtm9 closed this Oct 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants