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

Oxford collection + phpcs fixer cleanup. #22

Merged
merged 1 commit into from
Oct 29, 2015

Conversation

defrag
Copy link
Member

@defrag defrag commented Oct 28, 2015

No description provided.

'comma_separated',
array("%list%" => 'Michal, Norbert', '%last%' => "Lukasz"),
'oxford'
)->willReturn('Michal, Norbert and Lukasz');

Choose a reason for hiding this comment

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

But where is the oxford comma?

Copy link
Member Author

Choose a reason for hiding this comment

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

Ur right, but some ppl in different communites always asked for ability to specify if you need comma or not, so the format is defined in translation file. In that case everyone can adjust this to their own locale. Tho maybe I should add the default comma in translation file.

Updating comma

Updating readme
norberttech pushed a commit that referenced this pull request Oct 29, 2015
Oxford collection +  phpcs fixer cleanup.
@norberttech norberttech merged commit 53ebf13 into coduo:master Oct 29, 2015
@@ -0,0 +1,3 @@
only_two: "%first% and %second%"
comma_separated: "%list%, and %last%"
comma_separated_with_limit: "{1} %list%, and 1 other|[2,Inf] %list%, and %count% others"
Copy link
Member

Choose a reason for hiding this comment

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

@orestes @lightglitch @dagaa @Forst @sarelvdwalt @ozmodiar @mattallty @cnkt @tbreuss @IgorDePaula @omissis - anyone willing to help us here with translations?

Copy link
Contributor

Choose a reason for hiding this comment

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

Will do :) This one seems pretty straightforward, no caveats as in difference.xx.yaml
I could of course point out that many languages have gender-depending noun forms and in this case it affects "1 other", but this can be worked around.

Copy link
Member

Choose a reason for hiding this comment

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

@Forst awesome, thanks!

Copy link

Choose a reason for hiding this comment

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

Sure. No problem with Spanish, Galician or Catalan.

Copy link
Contributor

Choose a reason for hiding this comment

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

I've just sent in the PR containing the italian translation.

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.

6 participants