feat(locale): add Tamil language support#3468
feat(locale): add Tamil language support#3468xDivisionByZerox merged 6 commits intofaker-js:nextfrom
Conversation
✅ Deploy Preview for fakerjs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
matthewmayer
left a comment
There was a problem hiding this comment.
- We ask in the initial PR for a new locale that you only include basic metadata, plus a single module of locale data (e.g.
date) - that makes it easier to review. You can then follow up with additional PRs once approved - Given for example the city list includes only Indian cities and the phone numbers start +91, this should probably be ta_IN as Tamil is also spoken in e.g. Sri Lanka and Singapore.
- The script code in metadata should be from https://en.wikipedia.org/wiki/ISO_15924 ie
TamlnotTamil
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## next #3468 +/- ##
=======================================
Coverage 99.97% 99.97%
=======================================
Files 2819 2825 +6
Lines 217500 217571 +71
Branches 953 953
=======================================
+ Hits 217445 217518 +73
+ Misses 55 53 -2
🚀 New features to boost your workflow:
|
matthewmayer
left a comment
There was a problem hiding this comment.
LGTM. Please allow a few days for the maintainers to review.
xDivisionByZerox
left a comment
There was a problem hiding this comment.
Thank you for your contribution. I'll merge this PR, so feel free to contribute the other modules you initially provided as well. Remember, try to keep each PR to one module. This helps us in the review process as well as you and the community since your will PRs will get merged faster. 🙂
|
@Pradeepg2445 I see that this is your first contribution to our project.
|
|
Thank you so much for the warm message. It really means a lot to me as a first-time contributor ✨. What motivated me: Did I read the docs: Before the PR: When I was stuck: Code review: Anything else: |
|
@Pradeepg2445 are you still interested in contributing some further modules for ta_IN? |
|
Hi @matthewmayer, Yes, I’m definitely still interested in contributing more modules for ta_IN. I’ve also started preparing the variants for ta_LK (Sri Lanka Tamil) and ta_SG (Singapore Tamil) as separate locale codes. Sorry for the slight delay, I’ve been handling a few things recently, but I’ll submit the next modules within the coming days. Thanks again for the guidance and support! |
Added Tamil (ta) localization support including translations for address, person, and internet modules.
pnpm run preflightpassed