Skip to content

Merge from master #81

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
ikostan merged 13 commits into from
Sep 1, 2025
Merged

Merge from master #81

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
6882938
[Sync Iteration] python/pig-latin/9
exercism-solutions-syncer[bot] Sep 1, 2025
cb5ebfa
[Sync Iteration] python/pig-latin/10
exercism-solutions-syncer[bot] Sep 1, 2025
f171e0d
Update pig_latin.py
ikostan Sep 1, 2025
ea28217
Update matching_brackets_test.py
ikostan Sep 1, 2025
8c21c34
Update matching_brackets_test.py
ikostan Sep 1, 2025
b8de9ac
Del brackets
ikostan Sep 1, 2025
45143cb
Update pig_latin.py
ikostan Sep 1, 2025
ea28e94
[Sync Iteration] python/pig-latin/13
exercism-solutions-syncer[bot] Sep 1, 2025
6d8030a
Merge pull request #78 from ikostan/exercism-sync/2f6a7f64ce634abf
ikostan Sep 1, 2025
9a0d5b8
Merge pull request #79 from ikostan/main
ikostan Sep 1, 2025
0a97dbb
Merge pull request #74 from ikostan/exercism-sync/c3a868b4d0869f4e
ikostan Sep 1, 2025
ce437f3
Merge pull request #80 from ikostan/main
ikostan Sep 1, 2025
afa9f92
Merge pull request #75 from ikostan/exercism-sync/45b491e98548b713
ikostan Sep 1, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
32 changes: 0 additions & 32 deletions matching-brackets/.exercism/config.json
View file
Open in desktop

This file was deleted.

1 change: 0 additions & 1 deletion matching-brackets/.exercism/metadata.json
View file
Open in desktop

This file was deleted.

130 changes: 0 additions & 130 deletions matching-brackets/HELP.md
View file
Open in desktop

This file was deleted.

44 changes: 0 additions & 44 deletions matching-brackets/README.md
View file
Open in desktop

This file was deleted.

2 changes: 0 additions & 2 deletions matching-brackets/matching_brackets.py
View file
Open in desktop

This file was deleted.

76 changes: 0 additions & 76 deletions matching-brackets/matching_brackets_test.py
View file
Open in desktop

This file was deleted.

18 changes: 4 additions & 14 deletions pig-latin/pig_latin.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,10 +17,10 @@ def translate(text: str) -> str:
:return: The text translated to Pig Latin
"""
words: list[str] = text.split(" ")
return " ".join(process_words(word) for word in words)
return " ".join(process_word(word) for word in words)


def process_words(text: str) -> str:
def process_word(text: str) -> str:
"""
Process a single word and convert it to Pig Latin using the four translation rules.

Expand Down Expand Up @@ -54,7 +54,7 @@ def process_words(text: str) -> str:
return f"{text[i:]}{text[:i]}ay"

# Rule 2
if is_consonant(text[0]):
if not is_vowel(text[0]):
# If a word begins with one or more consonants, first move those consonants
# to the end of the word and then add an "ay" sound to the end of the word.
i = get_consonant_cluster_length(text)
Expand Down Expand Up @@ -106,16 +106,6 @@ def is_vowel(char: str) -> bool:
return char in "aeiou"


def is_consonant(char: str) -> bool:
"""
Check if a character is a consonant (one of the 21 non-vowel letters).

:param char: The character to check
:return: True if the character is a consonant, False otherwise
"""
return char.isalpha() and not is_vowel(char)


def get_consonant_cluster_length(text: str) -> int:
"""
Find the length of the consonant cluster at the beginning of a word.
Expand All @@ -125,7 +115,7 @@ def get_consonant_cluster_length(text: str) -> int:
"""
i = 0
for n, char in enumerate(text):
if is_consonant(char):
if not is_vowel(char):
i = n
else:
break
Expand Down
Loading