-
Notifications
You must be signed in to change notification settings - Fork 394
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
chore: compile messages to json #1888
Conversation
@aseerkt is attempting to deploy a commit to the lingui Team on Vercel. A member of the Team first needs to authorize it. |
size-limit report 📦
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1888 +/- ##
==========================================
+ Coverage 76.63% 76.65% +0.01%
==========================================
Files 81 81
Lines 2089 2090 +1
Branches 534 533 -1
==========================================
+ Hits 1601 1602 +1
Misses 375 375
Partials 113 113 ☔ View full report in Codecov by Sentry. |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
case "es": | ||
ext = "mjs" | ||
break | ||
case "ts": |
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.
Did you forget to port the "ts" case?
ext = "ts"
break
I'm surprised that the test doesn't catch it!
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.
Ah nevermind I just understood what you did!
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.
@aseerkt thank you!
Description
Compile catalogs to JSON
Types of changes
Fixes #1866
Checklist