Skip to content

test: migrate LeetCode 208 to new directory structure with tests#107

Merged
kobukuro merged 1 commit intomainfrom
test/migrate-leetcode-208-with-tests
Apr 8, 2026
Merged

test: migrate LeetCode 208 to new directory structure with tests#107
kobukuro merged 1 commit intomainfrom
test/migrate-leetcode-208-with-tests

Conversation

@kobukuro
Copy link
Copy Markdown
Owner

@kobukuro kobukuro commented Apr 8, 2026

Migrate Trie (Prefix Tree) implementation from old directory structure to standardized problems/medium/ layout following project conventions.

Code improvements:

  • Remove unnecessary is_end parameter from init
  • Add type hints (dict[str, Trie]) for better clarity
  • Simplify variable naming (curr_node → curr, w → c)
  • Follow PEP 8 naming convention (startsWith → starts_with)

Documentation:

  • Move complexity comments into proper docstrings
  • Update README.md: increment total count (87→88) and medium count (25→26)
  • Update docs/MEDIUM.md: add problem 208 entry

Test coverage:

  • LeetCode official example test case

  Migrate Trie (Prefix Tree) implementation from old directory structure to standardized problems/medium/ layout following project conventions.

  Code improvements:
  - Remove unnecessary is_end parameter from __init__
  - Add type hints (dict[str, Trie]) for better clarity
  - Simplify variable naming (curr_node → curr, w → c)
  - Follow PEP 8 naming convention (startsWith → starts_with)

  Documentation:
  - Move complexity comments into proper docstrings
  - Update README.md: increment total count (87→88) and medium count (25→26)
  - Update docs/MEDIUM.md: add problem 208 entry

  Test coverage:
  - LeetCode official example test case
@kobukuro kobukuro merged commit bdccc10 into main Apr 8, 2026
1 check passed
@kobukuro kobukuro deleted the test/migrate-leetcode-208-with-tests branch April 8, 2026 01:05
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.

1 participant