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

fa-IR locale with tests plus 1 fix for gl locale #979

Closed
wants to merge 6 commits into from

Conversation

pkhodaveissi
Copy link

@pkhodaveissi pkhodaveissi commented Nov 7, 2018

Please review and let me know what could be next

fixed #978

Copy link
Member

@kossnocorp kossnocorp left a comment

Choose a reason for hiding this comment

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

The PR is looking great, thank you for your work! Please remove the unnecessary comment and redundant .DS_Store files (I recommend you to echo .DS_Store >> ~/.gitignore_global). Right after that, I'll ship it!

src/locale/fa-IR/_lib/localize/index.js Outdated Show resolved Hide resolved
src/locale/gl/_lib/formatRelative/index.js Show resolved Hide resolved
@pkhodaveissi
Copy link
Author

I did all you asked for, let me know what's wrong

@kossnocorp
Copy link
Member

@pkhodaveissi sorry, I don't have enough time to do a proper review right now. It would help to move things forward if you also update match/index.js because I won't be able to publish the locale without it.

@kossnocorp
Copy link
Member

Closed by #1113

@kossnocorp kossnocorp closed this Jun 5, 2019
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.

Persian language support
2 participants