Conversation
New languages: Japanese, Korean, Arabic, Italian, Dutch, Turkish, Polish, Swedish, Indonesian, Thai, Norwegian, Finnish, Icelandic. Full Scandinavian coverage (Danish, Swedish, Norwegian, Finnish, Icelandic). All languages support BigInt up to 10^36. 265 tests passing. Version bumped to 1.0.1. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 603107396b
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| const hundredIt = (n) => { | ||
| if (n < 100 || n >= 1000) return ''; | ||
| const h = Math.floor(n / 100); | ||
| return IT_HUNDREDS[h]; |
There was a problem hiding this comment.
Handle cento elision before otto/ottanta
Italian drops the trailing “o” in cento before words starting with otto (e.g., 108 → “centotto”, 180 → “centottanta”). Here hundredIt always returns cento, and the later concatenation with tenIt(8) or tenIt(80) produces “centootto/centoottanta” instead. This makes all numbers 108/180/181…189 incorrect in the new Italian implementation.
Useful? React with 👍 / 👎.
Summary
New Language Files
jalanguages/ja.jskolanguages/ko.jsarlanguages/ar.jsitlanguages/it.jsnllanguages/nl.jstrlanguages/tr.jspllanguages/pl.jssvlanguages/sv.jsidlanguages/id.jsthlanguages/th.jsnolanguages/no.jsfilanguages/fi.jsislanguages/is.jsTest plan
npm run lint— zero errorsnpm test— 265/265 passingtoWords()npm publishas v1.0.1🤖 Generated with Claude Code