-
-
Notifications
You must be signed in to change notification settings - Fork 4
Use flit-gettext to compile gettext in distributions #166
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR migrates the build system from flit_core/flit_scm to flit-gettext with SCM support to enable automatic compilation of gettext translation files during the package build process. The project contains German translations in .po files that need to be compiled.
- Replaces
flit_coreandflit_scmwithflit-gettext[scm]in the build system configuration - Updates the build backend module path from
flit_scm:buildapitoflit_gettext.scm - Adds gettext system dependency installation to CI and release workflows
Reviewed Changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| pyproject.toml | Updates build system requirements and backend to use flit-gettext for translation support |
| .github/workflows/release.yml | Adds gettext installation step before building release artifacts |
| .github/workflows/ci.yml | Adds gettext installation step before building distribution packages |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
175ab88 to
1c37345
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@ ## main #166 +/- ## ========================================== - Coverage 99.69% 99.41% -0.29% ========================================== Files 21 21 Lines 332 343 +11 ========================================== + Hits 331 341 +10 - Misses 1 2 +1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
No description provided.