Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[Fix #401 ext] Return localized labels in month #508

Merged
merged 3 commits into from
Jan 18, 2017

Conversation

alberthkcheng
Copy link
Contributor

Return localized month in the same way as weekday.

@alberthkcheng
Copy link
Contributor Author

pls check again the comment in #401 . thanks :)

@vspinu
Copy link
Member

vspinu commented Jan 16, 2017

Thanks, but this is not backward compatible, but I guess it's the right way to go anyways.

Could you please add a note to NEWS.md and check if there are tests for english locale?

@alberthkcheng
Copy link
Contributor Author

NEW.md and tests are added for month and wday

@vspinu vspinu merged commit ebd90d9 into tidyverse:master Jan 18, 2017
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants