Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Polishordinals #367

Merged
merged 1 commit into from Apr 23, 2021
Merged

Conversation

PaulinaKomorek
Copy link
Contributor

polish ordinals # by PaulinaKomorek

Changes proposed in this pull request:

  • added support for polish ordinal numbers

Status

  • READY
  • HOLD
  • WIP (Work-In-Progress)

How to verify this change

unit tests cover all of the possible cases

Additional notes

-

@coveralls
Copy link

coveralls commented Feb 7, 2021

Pull Request Test Coverage Report for Build 1015

  • 56 of 57 (98.25%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.002%) to 95.903%

Changes Missing Coverage Covered Lines Changed/Added Lines %
num2words/lang_PL.py 42 43 97.67%
Totals Coverage Status
Change from base Build 1008: 0.002%
Covered Lines: 6149
Relevant Lines: 6332

💛 - Coveralls

@PaulinaKomorek
Copy link
Contributor Author

@mromdhane
Could I kindly ask you for a review?

@mromdhane mromdhane self-requested a review March 1, 2021 14:59
@mromdhane
Copy link
Contributor

mromdhane commented Mar 4, 2021

@PaulinaKomorek Can you rebase/Squash commits into one please. I am reviewing it .
And can you add a commit message more clear [ADD] polish ordinal numbers

@PaulinaKomorek PaulinaKomorek force-pushed the polishordinals branch 3 times, most recently from 48b2806 to a3f247f Compare March 4, 2021 21:04
@PaulinaKomorek
Copy link
Contributor Author

@mromdhane Sorry, I am afraid I am not experienced enough in git, so I was unable to rebase it successfully, instead. I created a new commit with the proper message and all changes and force-pushed it into this branch. I hope that it is enough.

@PaulinaKomorek
Copy link
Contributor Author

Hello, @mromdhane any update on this topic?

@mromdhane
Copy link
Contributor

Hi @PaulinaKomorek Yes I check it, Can you merge master with your branch, please?

@PaulinaKomorek
Copy link
Contributor Author

@mromdhane Done. Thank you!

@mromdhane mromdhane self-assigned this Apr 23, 2021
@mromdhane mromdhane merged commit 3f5f616 into savoirfairelinux:master Apr 23, 2021
KommuSoft pushed a commit to hapytex/num2words that referenced this pull request Aug 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants