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

Сorrect plural forms for cs and sk locales #373

Conversation

ykshatroff
Copy link

This PR changes plural forms for Czech (cs) and Slovak (sk) locales from Plural rule #7 to Plural rule #8 according to https://developer.mozilla.org/en-US/docs/Mozilla/Localization/Localization_and_Plurals .

Fixes #372

Change plural forms for Czech (cs) and Slovak (sk) locales from Plural rule #_7_ to Plural rule #_8_ according to https://developer.mozilla.org/en-US/docs/Mozilla/Localization/Localization_and_Plurals .

Fixes python-babel#372
@ykshatroff ykshatroff force-pushed the fix-issue-372-wrong-plurals-for-cs-and-sk branch from e811a35 to 128afa0 Compare April 5, 2016 09:41
@codecov-io
Copy link

Current coverage is 89.10%

Merging #373 into master will decrease coverage by -0.65% as of 076d66f

@@            master   #373   diff @@
=====================================
  Files           23     23       
  Stmts         3854   3854       
  Branches         0      0       
  Methods          0      0       
=====================================
- Hit           3459   3434    -25
  Partial          0      0       
- Missed         395    420    +25

Review entire Coverage Diff as of 076d66f

Powered by Codecov. Updated on successful CI builds.

@akx
Copy link
Member

akx commented Jul 8, 2016

Thanks for the Mozilla ref! I think we'll have to take a look at that list of rules at some point; there's bound to be something else that's not quite correct still.

@akx akx merged commit ad20a72 into python-babel:master Jul 8, 2016
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.

None yet

4 participants