-
-
Notifications
You must be signed in to change notification settings - Fork 929
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
feat: migrate locales to typescript #238
Conversation
✔️ Deploy Preview for vigilant-wescoff-04e480 ready! 🔨 Explore the source changes: 58b1cd1 🔍 Inspect the deploy log: https://app.netlify.com/sites/vigilant-wescoff-04e480/deploys/61e940d03e58cc0007a644d3 😎 Browse the preview: https://deploy-preview-238--vigilant-wescoff-04e480.netlify.app |
As I yesterday evening said in Discord, we need to fix the types and maybe some keys later in another PR |
@Shinigami92: Reverted |
Seems there is an issue with |
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 worked together with @ST-DDT via screenshare on this PR
I briefly scanned all commits again and compared the data tree again and found a typo in one of my manual changes. |
It's past 12 o'clock, so the one fixed random test fails on windows. |
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.
diff and test looks good
Ps.: did you write the script in Java? :D
I automatically transformed the locales using the following
java
script.Source (Click to expand)
Faker
checkout.java Convert.java
using java 17Fix type errors(Reverted as requested in feat: migrate locales to typescript #238 (comment))I recommend reviewing the PR commit by commit instead of all at once.
Tests that nothing has been changed
via
pnpm esno index.ts > output_old.json
via
pnpm esno index.ts > output_new.json
Accidental fixes
faker/src/locales/hr/index.ts
Line 9 in 66aa374