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

Support for Ukrainian (uk) #519

Merged
merged 5 commits into from
Jul 13, 2023
Merged

Support for Ukrainian (uk) #519

merged 5 commits into from
Jul 13, 2023

Conversation

bootseller
Copy link
Contributor

  • added support for Ukrainian
  • added the corresponding tests

@wanasit
Copy link
Owner

wanasit commented Jul 8, 2023

Hello, @bootseller. Thanks for the improvement and sorry for my slow reply.

I'll review this today, but note that it may take time because the change is large. Next time, I'd recommend breaking the changes into smaller pieces (e.g. just minimum config, one pattern at a time) rather than submitting the all pattern translation at once.

Anyway thanks again for your help.

Copy link
Owner

@wanasit wanasit left a comment

Choose a reason for hiding this comment

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

Overall. lgtm.

There is only one minor comment. I'll merge it this weekend.

src/locales/uk/index.ts Show resolved Hide resolved
@wanasit
Copy link
Owner

wanasit commented Jul 8, 2023

The automated test failed because of the package config in the main branch. If you could rebase to the latest master, the test should pass.

@bootseller
Copy link
Contributor Author

Thank you so much for looking into my PR. I think I did everything you asked for.

Do let me know if I need to implement anything else.

@vkartaviy
Copy link

@bootseller Thank you! 🇺🇦🇺🇦🇺🇦

@coveralls
Copy link

Coverage Status

coverage: 91.355% (+0.2%) from 91.139% when pulling 664fe87 on bootseller:master into 2930624 on wanasit:master.

@wanasit wanasit merged commit 7b2ef04 into wanasit:master Jul 13, 2023
@wanasit
Copy link
Owner

wanasit commented Jul 13, 2023

This change has been included in v2.6.4.

@bootseller
Copy link
Contributor Author

Thank you!

@wanasit wanasit mentioned this pull request Sep 2, 2023
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.

4 participants