Skip to content

Conversation

@ABolouk
Copy link
Contributor

@ABolouk ABolouk commented Dec 1, 2025

Pull Request Checklist

Thank you for taking the time to improve Arrow! Before submitting your pull request, please check all appropriate boxes:

  • 🧪 Added tests for changed code.
  • 🛠️ All tests pass when run locally (run tox or make test to find out!).
  • 🧹 All linting checks pass when run locally (run tox -e lint or make lint to find out!).
  • 📚 Updated documentation for changed code.
  • ⏩ Code is up-to-date with the master branch.

If you have any questions about your code changes or any of the points above, please submit your questions along with the pull request and we will try our best to help!

Description of Changes

Closes: #1022

Added test cases for the languages that were not in in the test_locale file based on their related Locale class data.

@codecov
Copy link

codecov bot commented Dec 1, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (master@edfff3c). Learn more about missing BASE report.
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##             master     #1236   +/-   ##
==========================================
  Coverage          ?   100.00%           
==========================================
  Files             ?        10           
  Lines             ?      2315           
  Branches          ?       358           
==========================================
  Hits              ?      2315           
  Misses            ?         0           
  Partials          ?         0           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@ABolouk
Copy link
Contributor Author

ABolouk commented Dec 1, 2025

I just want to mention for Romansh and Catalan languages although there are translation for "week" and "weeks" but it seems that they can not generate translation for "weeks".
I removed the tests related to "weeks" for these two so that there wouldn't be a test for something not implemented.
I would be glad to know if this is okay or do I need to add them.

Thank you.

@jadchaar
Copy link
Member

jadchaar commented Dec 7, 2025

Hey @ABolouk, mind fixing the linting as well?

@jadchaar jadchaar merged commit 598b72a into arrow-py:master Dec 15, 2025
27 checks passed
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.

More Detailed Test Cases For Locales

2 participants