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

collection formatter generalized to "OxfordStyle" and "RegularStyle" #386

Closed
wants to merge 3 commits into from
Closed

collection formatter generalized to "OxfordStyle" and "RegularStyle" #386

wants to merge 3 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Feb 25, 2015

This change is regarding the concatenation of multiple items with comma and e.g. "and". See:

new []{"Peter", "Paul", "Mary"} => EN => "Peter, Paul, and Mary" (if you use Oxford style)
new []{"Peter", "Paul", "Mary"} => EN => "Peter, Paul and Mary" (if you dont use Oxford style)
new []{"Peter", "Paul", "Mary"} => DE => "Peter, Paul and Mary" (this is how it works in italian and german)

The English and Italian version got generaltized to "OxfordSyle" and "RegularStyle". The registry now uses these base classes with a constructor parameter to get the default separator.
The registry now registers a german collection formatter.

Thomas Mentzel added 2 commits February 25, 2015 16:18
…(regarding the concatenation and 'comma before and), registry uses for 'de' and 'it' the regular style, for 'en' still the oxford style.
@hazzik
Copy link
Member

hazzik commented Feb 25, 2015

Actually this is a good change after close look. Sorry about that

@mexx
Copy link
Collaborator

mexx commented Feb 26, 2015

@ThomasMentzel please add this PR to release notes according to CONTRIBUTING.md with message like "added German collection formatter".

@ghost
Copy link
Author

ghost commented Feb 27, 2015

I added the required information. I added additional languages according to wikipedia and google translator .

@MehdiK
Copy link
Member

MehdiK commented Feb 28, 2015

Very nice addition. Thanks. This is now rebased and pushed.

@MehdiK
Copy link
Member

MehdiK commented Mar 4, 2015

This is now released to NuGet as v1.34.0. Thanks.

@Bartmax
Copy link

Bartmax commented Mar 29, 2015

DefaultDeparator ??

@MehdiK
Copy link
Member

MehdiK commented Mar 29, 2015

Sorry @Bartmax - not sure what you mean! Can you please elaborate a bit?

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