Skip to content

feat(python): add locale script and more c deps#4

Merged
ReenigneArcher merged 4 commits into
masterfrom
feat/python/add-locale-script-and-more-c-deps
Jun 4, 2026
Merged

feat(python): add locale script and more c deps#4
ReenigneArcher merged 4 commits into
masterfrom
feat/python/add-locale-script-and-more-c-deps

Conversation

@ReenigneArcher
Copy link
Copy Markdown
Member

Description

Adds python _locale.py script. This should be the final thing necessary to drop individual python setups from c/c++ projects (LizardByte/roadmap#161).

Screenshot

Issues Fixed or Closed

Roadmap Issues

Type of Change

  • feat: New feature (non-breaking change which adds functionality)
  • fix: Bug fix (non-breaking change which fixes an issue)
  • docs: Documentation only changes
  • style: Changes that do not affect the meaning of the code (white-space, formatting, missing semicolons, etc.)
  • refactor: Code change that neither fixes a bug nor adds a feature
  • perf: Code change that improves performance
  • test: Adding missing tests or correcting existing tests
  • build: Changes that affect the build system or external dependencies
  • ci: Changes to CI configuration files and scripts
  • chore: Other changes that don't modify src or test files
  • revert: Reverts a previous commit
  • BREAKING CHANGE: Introduces a breaking change (can be combined with any type above)

Checklist

  • Code follows the style guidelines of this project
  • Code has been self-reviewed
  • Code has been commented, particularly in hard-to-understand areas
  • Code docstring/documentation-blocks for new or existing methods/components have been added or updated
  • Unit tests have been added or updated for any new or modified functionality

AI Usage

  • None: No AI tools were used in creating this PR
  • Light: AI provided minor assistance (formatting, simple suggestions)
  • Moderate: AI helped with code generation or debugging specific parts
  • Heavy: AI generated most or all of the code changes

@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 4, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (ad923ba) to head (3cbd34a).
✅ All tests successful. No failed tests found.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##            master        #4    +/-   ##
==========================================
  Coverage   100.00%   100.00%            
==========================================
  Files            1         2     +1     
  Lines           14       163   +149     
==========================================
+ Hits            14       163   +149     
Flag Coverage Δ
Linux 100.00% <100.00%> (ø)
Windows 100.00% <100.00%> (ø)
macOS 100.00% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
scripts/localize.py 100.00% <100.00%> (ø)
scripts/update_clang_format.py 100.00% <ø> (ø)

Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ad923ba...3cbd34a. Read the comment docs.

@ReenigneArcher ReenigneArcher force-pushed the feat/python/add-locale-script-and-more-c-deps branch from 7dab75a to b2e3916 Compare June 4, 2026 17:51
@ReenigneArcher ReenigneArcher force-pushed the feat/python/add-locale-script-and-more-c-deps branch from b2e3916 to c0cfc46 Compare June 4, 2026 17:56
@ReenigneArcher ReenigneArcher marked this pull request as ready for review June 4, 2026 17:58
Add .github/workflows/localize.yml to run the shared locale extractor (.lizardbyte-common), sync uv, extract gettext/Babel strings, and open localization update PRs. Update README to document scripts/_locale.py, use of uv sync --locked, example workflow usage, and commands to run gettext extraction and tests.
Rename scripts/_locale.py to scripts/localize.py and update all references. Adjust README examples, the GitHub Actions workflow (.github/workflows/localize.yml) to call the new script path, and rename tests/unit/test_locale.py to tests/unit/test_localize.py with imports and constant references updated accordingly.
Add a step that extracts workflow_repository and workflow_sha from the job context (using jq on toJSON(job)), validates them, and exposes them as step outputs via GITHUB_OUTPUT. Update the subsequent actions/checkout for .lizardbyte-common to use those outputs (steps.shared-workflow.outputs.repository and .sha) instead of job.workflow_repository/job.workflow_sha. This ensures the reusable/shared workflow repository and commit are resolved reliably before performing the checkout.
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Jun 4, 2026

@ReenigneArcher ReenigneArcher merged commit fd6c6b4 into master Jun 4, 2026
16 checks passed
@ReenigneArcher ReenigneArcher deleted the feat/python/add-locale-script-and-more-c-deps branch June 4, 2026 19:09
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.

1 participant