savoirfairelinux / num2words

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

GNU Lesser General Public License v2.1

820 stars 497 forks source link

#454: [tr] fix negative number problem #477

Closed emregeldegul closed 2 years ago

emregeldegul commented 2 years ago

454 Negative Number return wrong text Fixes @emregeldegul

Changes proposed in this pull request:

-50 (cardinal) -> eksielli
-3 (currency) -> eksiellilira

Status

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

How to verify this change

Add new cases in the turkish test for negative numbers

Additional notes

I opened a PR before but closed it because there were too many conflicts.

mrodriguezg1991 commented 2 years ago

@emregeldegul can you fix this for python3.10?

   flake8 run-test: commands[0] | flake8
  ./tests/test_tr.py:192:80: E501 line too long (82 > 79 characters)

emregeldegul commented 2 years ago

@emregeldegul can you fix this for python3.10?

   flake8 run-test: commands[0] | flake8
  ./tests/test_tr.py:192:80: E501 line too long (82 > 79 characters)

i fixed it.

coveralls commented 2 years ago

Coverage increased (+0.002%) to 96.555% when pulling a7d095c97db49c9583af02655676f6c1d82332f5 on emregeldegul:#454 into a218b332d25d716ea45acdd7f1f50ef9e5ad1e7a on savoirfairelinux:master.