Skip to content

[WIP][Translations][tests] Tests for PluralizationRules. #4412

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

Merged
merged 1 commit into from
Jul 9, 2012

Conversation

clemens-tolboom
Copy link
Contributor

Currently we have no tests PluralizationRules. This patch is an initial one to show we have not enough langcodes in PluralizationRules.

I hope this gets in so others can fix the missing langcodes.

I'm waiting for RFC [MessageCatalogue*] Add Metadata to MessageCatalogue to get in to continue working on the [WIP]: Allow Drupal to use Translate component.

@travisbot
Copy link

This pull request passes (merged 3462afc into 023dbf8).

@ghost
Copy link

ghost commented Jul 1, 2012

Is there anything pending in this PR?

*
* @return array with nplural together with langcodes
*/
public function failingLangcodes()
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I would say someone should check the failingLangcodes().

fabpot added a commit that referenced this pull request Jul 9, 2012
Commits
-------

3462afc Tests for PluralizationRules.

Discussion
----------

[WIP][Translations][tests] Tests for PluralizationRules.

Currently we have no tests PluralizationRules. This patch is an initial one to show we have not enough langcodes in PluralizationRules.

I hope this gets in so others can fix the missing langcodes.

I'm waiting for [RFC [MessageCatalogue*] Add Metadata to MessageCatalogue](#4399) to get in to continue working on the [[WIP]: Allow Drupal to use Translate component](#4249).

---------------------------------------------------------------------------

by travisbot at 2012-05-25T14:38:37Z

This pull request [passes](http://travis-ci.org/symfony/symfony/builds/1433558) (merged 3462afc into 023dbf8).

---------------------------------------------------------------------------

by drak at 2012-07-01T09:47:05Z

Is there anything pending in this PR?
@fabpot fabpot merged commit 3462afc into symfony:master Jul 9, 2012
@clemens-tolboom clemens-tolboom deleted the pluraltests branch December 23, 2013 10:05
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.

3 participants