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 Persian language with code fa-IR #903
Conversation
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.
Thank you for your work. Please address my comments.
import assert from 'power-assert' | ||
import formatDistance from '.' | ||
|
||
describe('en-US locale > formatDistance', function () { |
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.
Please update the description
assert(result === 'in half a minute') | ||
}) | ||
}) | ||
}) |
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.
These tests aren't implemented.
// Note: in English, the names of days of the week and months are capitalized. | ||
// If you are making a new locale based on this one, check if the same is true for the language you're working on. | ||
// Generally, formatted dates should look like they are in the middle of a sentence, | ||
// e.g. in Spanish language the weekdays and months should be in the lowercase. |
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.
Please remove the comment as it's not relevant anymore.
// var unit = String(options.unit) | ||
// | ||
// where `unit` can be 'year', 'quarter', 'month', 'week', 'date', 'dayOfYear', | ||
// 'day', 'hour', 'minute', 'second' |
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.
Please remove the comment, it's not relevant anymore.
* @iso-639-2 eng | ||
* @author Sasha Koss [@kossnocorp]{@link https://github.com/kossnocorp} | ||
* @author Lesha Koss [@leshakoss]{@link https://github.com/leshakoss} | ||
*/ |
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.
Please update the JSDoc.
import formatRelative from '../../formatRelative' | ||
import parse from '../../parse' | ||
|
||
describe('en-US locale', function () { |
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.
Description
assert.deepEqual(result, new Date(1986, 3 /* Apr */, 4, 1, 2, 3)) | ||
}) | ||
}) | ||
}) |
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.
Also in progress?
@MahdiMajidzadeh Ping. |
Closed by #1113 |
No description provided.