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

Languages plurals #108

Merged
merged 3 commits into from
Nov 30, 2018
Merged

Languages plurals #108

merged 3 commits into from
Nov 30, 2018

Conversation

daliusd
Copy link
Contributor

@daliusd daliusd commented Nov 16, 2018

Improved languages plurals support.

@coveralls
Copy link

coveralls commented Nov 16, 2018

Coverage Status

Coverage increased (+0.3%) to 91.813% when pulling bd501f8 on daliusd:languages-plurals into 4287028 on i18next:master.

avreg and others added 2 commits November 23, 2018 09:14
* Only v3 support left. There is no reason to add non working code.

* Obejct used instead of array. Array is working because of wrong
reasons.

* Instead of failing all process because of one wrong language message
is written to log.

* Test coverage increased.
@daliusd
Copy link
Contributor Author

daliusd commented Nov 29, 2018

@cheton is there anything I could do on this pull request to get it accepted?

@cheton cheton merged commit a81bac7 into i18next:master Nov 30, 2018
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