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

Added Tajik language support #406

Merged
merged 5 commits into from
Aug 9, 2022
Merged

Added Tajik language support #406

merged 5 commits into from
Aug 9, 2022

Conversation

decherd
Copy link
Contributor

@decherd decherd commented Jul 2, 2021

Fixes # by Drew Echerd

Changes proposed in this pull request:

  • Added Tajik language support and tests

Status

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

How to verify this change

*All tests pass
*Google translate isn't 100% accurate for Tajik. Tajik is a Persian language but the only one that uses Cyrillic. There isn't a similar language file.
*Trust me.

Additional notes

@decherd
Copy link
Contributor Author

decherd commented Jul 2, 2021

@mromdhane
Would you mind reviewing this PR? I believe everything should be good with flake8 and test coverage.

@decherd
Copy link
Contributor Author

decherd commented Jul 3, 2021

The code I added for Tajik support is at 100% coverage. None of the checks completed. Are those supposed to run automatically or is a review required first? This is my first pull request. I'm pretty excited to both be contributing to my first open source project and to be adding Tajik language support. I've attempted to make sure I'm not missing anything but if I am just let me know what I need to do. Thanks!

@mromdhane mromdhane self-requested a review July 6, 2021 13:45
@mromdhane
Copy link
Contributor

@decherd Thank you for your contribution ! we wait for the coverage and the tests after I will review your PR.

@decherd
Copy link
Contributor Author

decherd commented Jul 13, 2021

@mromdhane Is there a typical timeline of when the pull request should be reviewed and merged? Again, this is my first contribution to an open source project and didn't know how long I should be waiting for this. Thanks.

@mrodriguezg1991
Copy link
Contributor

@decherd Hello and thanks for your contribution, can you fix your PR to pass the test on python3.10 ?

@decherd
Copy link
Contributor Author

decherd commented Aug 9, 2022

I've fixed the flake8 errors and run tox locally and everything passes. I've been waiting over a year to see my contributions merged and would love if you could see that happen.

@coveralls
Copy link

Pull Request Test Coverage Report for Build 2825812230

  • 120 of 120 (100.0%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.06%) to 96.404%

Totals Coverage Status
Change from base Build 2799420615: 0.06%
Covered Lines: 6951
Relevant Lines: 7134

💛 - Coveralls

@mrodriguezg1991
Copy link
Contributor

I've fixed the flake8 errors and run tox locally and everything passes. I've been waiting over a year to see my contributions merged and would love if you could see that happen.

Sorry for the inconvenience, everything should be ok, this changes will be added in a new release soon

@mrodriguezg1991 mrodriguezg1991 merged commit fbf7dd4 into savoirfairelinux:master Aug 9, 2022
@decherd decherd deleted the add-tajik-lang branch August 9, 2022 19:25
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

4 participants