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

Added locale en-GB #563

Merged
merged 4 commits into from Sep 18, 2017
Merged

Added locale en-GB #563

merged 4 commits into from Sep 18, 2017

Conversation

glintik
Copy link
Contributor

@glintik glintik commented Sep 16, 2017

Hi,

Locale en-GB is added and tested.

@glintik glintik mentioned this pull request Sep 16, 2017
Copy link
Contributor

@leshakoss leshakoss left a comment

Choose a reason for hiding this comment

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

Good job! 👍 Please fix the code style issue and we're good to go

* @category Locales
* @summary English locale (United Kingdom).
* @language English
* @iso-639-2 eng
Copy link
Contributor

Choose a reason for hiding this comment

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

Please add @author tag, like this:

 * @author Alex [@glintik]{@link https://github.com/glintik}

Copy link
Contributor

@leshakoss leshakoss left a comment

Choose a reason for hiding this comment

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

Oops, missed a comment in the last review. This code style issue:

LT: 'HH:mm',
LTS: 'HH:mm:ss',
L: 'DD/MM/YYYY',
LL : 'D MMMM YYYY',
Copy link
Contributor

Choose a reason for hiding this comment

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

Please remove extra space after key here and in two following lines. https://travis-ci.org/date-fns/date-fns/jobs/276343759#L490-L492

@glintik
Copy link
Contributor Author

glintik commented Sep 17, 2017

Sorry, forgot to run eslint. Fixed.

Copy link
Contributor

@leshakoss leshakoss left a comment

Choose a reason for hiding this comment

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

👍

@leshakoss
Copy link
Contributor

@glintik while we're waiting for @kossnocorp to release this as a new version, could you please add @author tag to jsdoc like this: #563 (comment)

@kossnocorp kossnocorp merged commit 5e0b6b6 into date-fns:master Sep 18, 2017
kossnocorp added a commit that referenced this pull request Sep 18, 2017
@kossnocorp
Copy link
Member

It's released as v2.0.0-alpha.5: https://gist.github.com/kossnocorp/a307a464760b405bb78ef5020a4ab136#v200-alpha5.

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants