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

Migration zh-hant from v1 #417

Merged
merged 1 commit into from
Dec 24, 2021

Conversation

DingWeizhe
Copy link

Move ZH-Hant from v1

@DingWeizhe
Copy link
Author

DingWeizhe commented Dec 14, 2021

I have temporarily released the new package https://www.npmjs.com/package/chrono-node-v2-zh-preview
If you need to use it first, you can try it.
But when this PR is approved, I will set the package as deprecated.

@wanasit
Copy link
Owner

wanasit commented Dec 24, 2021

@DingWeizhe I'm very sorry for my slow reply.
Thank you for you improvement! I'm really appreciate you help working on this.

@wanasit
Copy link
Owner

wanasit commented Dec 24, 2021

Your change looks good to me. The only comment/concern I have is I'm not sure if zhHant (e.g. chrono.zhHant) is a common way to refer to the locale (e.g. I thought it should be chrono.zh and chrono.zh.hant)

I'm merging your change, but let me do some research about what is the common pattern for that first before publishing this.

@wanasit wanasit merged commit 0e56d72 into wanasit:master Dec 24, 2021
@wanasit
Copy link
Owner

wanasit commented Dec 24, 2021

Also. Happy Christmas! btw.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants