Skip to content

Added locale en-GB - #563

Merged
kossnocorp merged 4 commits into
date-fns:masterfrom
glintik:master
Sep 18, 2017
Merged

kossnocorp merged 4 commits into
date-fns:masterfrom
glintik:master

Conversation

@glintik

@glintik glintik commented Sep 16, 2017

Copy link
Copy Markdown
Contributor

Hi,

Locale en-GB is added and tested.

@glintik glintik mentioned this pull request Sep 16, 2017

@leshakoss leshakoss left a comment

Copy link
Copy Markdown
Member

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

Comment thread src/locale/en-GB/index.js
* @category Locales
* @summary English locale (United Kingdom).
* @language English
* @iso-639-2 eng

Copy link
Copy Markdown
Member

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}

@leshakoss leshakoss left a comment

Copy link
Copy Markdown
Member

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
Copy Markdown
Member

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

glintik commented Sep 17, 2017

Copy link
Copy Markdown
Contributor Author

Sorry, forgot to run eslint. Fixed.

@leshakoss leshakoss left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

👍

@leshakoss

Copy link
Copy Markdown
Member

@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
Copy Markdown
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.

4 participants