Skip to content

Port: en_IN — support arab/kharab scale [upstream #654]#31

Merged
jqueguiner merged 1 commit into
masterfrom
port/upstream-654-en-in-crore
May 1, 2026
Merged

Port: en_IN — support arab/kharab scale [upstream #654]#31
jqueguiner merged 1 commit into
masterfrom
port/upstream-654-en-in-crore

Conversation

@jqueguiner

Copy link
Copy Markdown
Owner

Ports savoirfairelinux/num2words#654 by @shrutiichandra.

Summary

Extends en_IN's Indian-numbering-system scale beyond crore:

Power Word
10⁵ lakh (existing)
10⁷ crore (existing)
10⁹ arab (new)
10¹¹ kharab (new)

Test plan

  • All 5 EN_IN tests still green
  • `9907781961` → "nine arab, ninety crore, seventy-seven lakh, eighty-one thousand, nine hundred and sixty-one"
  • `155215059811` → "one kharab, fifty-five arab, twenty-one crore, fifty lakh, fifty-nine thousand, eight hundred and eleven"

Credit

Original author: @shrutiichandra.

Original upstream PR: savoirfairelinux#654

🤖 Generated with Claude Code

Ports savoirfairelinux#654 by @shrutiichandra.

Extends en_IN's high-numword scale beyond crore so that very large
numbers in the Indian numbering system render correctly:
- 10^9  → "arab"
- 10^11 → "kharab"

Already-supported lakh (10^5) and crore (10^7) unchanged.

Original PR: savoirfairelinux#654

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@coveralls

Copy link
Copy Markdown

Coverage Report for CI Build 25195710932

Coverage decreased (-0.06%) to 94.298%

Details

  • Coverage decreased (-0.06%) from the base build.
  • Patch coverage: 2 of 2 lines across 1 file are fully covered (100%).
  • 5 coverage regressions across 1 file.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

5 previously-covered lines in 1 file lost coverage.

File Lines Losing Coverage Coverage
num2words2/init.py 5 87.5%

Coverage Stats

Coverage Status
Relevant Lines: 39523
Covered Lines: 37800
Line Coverage: 95.64%
Relevant Branches: 8338
Covered Branches: 7332
Branch Coverage: 87.93%
Branches in Coverage %: Yes
Coverage Strength: 5.74 hits per line

💛 - Coveralls

@jqueguiner
jqueguiner merged commit f3ede3f into master May 1, 2026
20 checks passed
@jqueguiner
jqueguiner deleted the port/upstream-654-en-in-crore branch May 1, 2026 00:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants