-
-
Notifications
You must be signed in to change notification settings - Fork 943
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
refactor(locale): fix truncated song names with commas #3327
Conversation
likely accidentally lost when originally imported ref faker-js#996 and https://manghammath.com/80s%20Hits/The%20Top%201000%20Songs%20of%20AllTime.pdf
✅ Deploy Preview for fakerjs ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site configuration. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## next #3327 +/- ##
==========================================
- Coverage 99.97% 99.96% -0.01%
==========================================
Files 2806 2806
Lines 217086 217086
Branches 975 966 -9
==========================================
- Hits 217032 217012 -20
- Misses 54 74 +20
|
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.
Why doesnt this cause snapshot updates? Are we just lucky?
yep, its mostly just slight renames so didn't affect the snapshotted values |
likely accidentally lost when originally imported
ref #996 and https://manghammath.com/80s%20Hits/The%20Top%201000%20Songs%20of%20AllTime.pdf