-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Add Spanish locale (closes #268) #269
Conversation
@@ -4,3 +4,4 @@ tmp | |||
dist/compressed | |||
test.espowered.js | |||
.envrc | |||
.idea/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd say you should put this into your global .gitignore
instead
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is ok, considering the number of JetBrains users.
@@ -0,0 +1,84 @@ | |||
var buildFormattingTokensRegExp = require('../../_lib/build_formatting_tokens_reg_exp/index.js') | |||
|
|||
function buildFormatLocale () { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Only one thing here: in Spanish, the months, week days and seasons's names normally are written with lower cases.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK, I will change it
@juanangosto could you please update the list of supported langs as well? https://github.com/date-fns/date-fns/blob/master/docs/i18n.md#supported-languages |
👍 |
@juanangosto Spanish language locale was released with 1.12.0. (diff: v1.11.2...v1.12.0) Thank you for your awesome contribution! |
No description provided.