Skip to content
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.

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

Added Russian (ru) language support #23

Merged
merged 3 commits into from
Aug 18, 2024
Merged

Conversation

kidig
Copy link
Contributor

@kidig kidig commented Aug 15, 2024

Added support for the Russian language, as time is typically spoken in Russian-speaking countries.

The issue is that in many Slavic languages, the word for "hour" also changes depending on the numeral.
I had to come up with something in the code. :)

@Proteusiq Proteusiq self-assigned this Aug 16, 2024
@Proteusiq
Copy link
Owner

Proteusiq commented Aug 16, 2024

This is very nice. We are now only missing the test. Could you duplicate English tests in test_numbers and test_clock and test if everything passes?

@Proteusiq Proteusiq added the enhancement New feature or request label Aug 16, 2024
@kidig
Copy link
Contributor Author

kidig commented Aug 18, 2024

Sure! Added some tests, and also fixed the one case :)

Copy link
Owner

@Proteusiq Proteusiq left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Beautiful. thank you 🙏

@Proteusiq Proteusiq merged commit 681387e into Proteusiq:main Aug 18, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants