savoirfairelinux / num2words

Modules to convert numbers to words. 42 --> forty-two
GNU Lesser General Public License v2.1
820 stars 497 forks source link

[ADD] to ordinal number for Turkish #468

Closed ieski closed 2 years ago

ieski commented 2 years ago

Added Turkish Ordinal Number # by...

Status

ieski commented 2 years ago

@mrodriguezg1991

thank you for your feedback.

sent again pull request

coveralls commented 2 years ago

Pull Request Test Coverage Report for Build 2883658952


Totals Coverage Status
Change from base Build 2876554493: 0.001%
Covered Lines: 7220
Relevant Lines: 7399

💛 - Coveralls
mrodriguezg1991 commented 2 years ago

@ieski Hello and thanks for the contribution The test coverage decreased , can you write some test to validate your change, and to increase the test coverage ? Thanks

ieski commented 2 years ago

@ieski Hello and thanks for the contribution The test coverage decreased , can you write some test to validate your change, and to increase the test coverage ? Thanks

ok. i sent it

thanks

mrodriguezg1991 commented 2 years ago

@ieski hello Can you fix the flake 8 issues on python3.10 ?

  flake8 run-test: commands[0] | flake8
  ./tests/test_tr.py:187:44: E231 missing whitespace after ','
  ./tests/test_tr.py:188:44: E231 missing whitespace after ','
  ./tests/test_tr.py:189:44: E231 missing whitespace after ','

Thanks