Adding Tetum Language - Githubissues

savoirfairelinux / num2words

Modules to convert numbers to words. 42 --> forty-two

GNU Lesser General Public License v2.1

820 stars 497 forks source link

Adding Tetum Language #576

Closed PeteCoward closed 4 weeks ago

PeteCoward commented 4 months ago

Changes proposed in this pull request:

Add Tetum language from Timor-Leste. https://en.wikipedia.org/wiki/Tetum_language

Status

[x] READY
[ ] HOLD
[] WIP (Work-In-Progress)

How to verify this change

Run the tests, or build and run num2words -l tet 101

coveralls commented 4 months ago

coverage: 97.864% (+0.07%) from 97.794% when pulling dfdfa0ce8142d79e06e025efde54c30764d48f5f on pykafe:lang_TET_base into ffebd91cdf19d1ffca0f8706ca764f3ea5e4e99b on savoirfairelinux:master.

PeteCoward commented 4 months ago

Hello, we are a bit puzzled by the coverage results we get here!

When we run too locally we get 100% coverage for the lang_TET.py file, and the detail links report 100% on that file too. This reports an increase in 3.12 and 3.7 and coveralls, but a decrease in 3.8, .3.9, 3.10, 3.11 and we do not understand why.

Can anyone help us?

mrodriguezg1991 commented 1 month ago

@PeteCoward Hello, can you update the PR and fix the error in the tests? Thanks for you contribution to the project