ci: update workflow to include localization dependencies and compile translations

SeedSigner/SeedSigner

3a449ab3 View source ↗

Public commit record

What the developer wrote

Authored by okaybro

62/100 · Adequate

ci: update workflow to include localization dependencies and compile translations

✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body

The short version

What changed, and why it matters

This commit updates the project's automated test workflow to install additional tools needed for handling translations and to compile those translations before running tests. It is a routine build/CI maintenance change with no apparent security relevance.

Recommended action

No security action required. Review as normal CI maintenance.

Security signals we found

No strong security signals were identified.

Technical analysis

Evidence from the diff

The commit modifies .github/workflows/tests.yml to add l10n/requirements-l10n.txt to the pip install step, switch pip install . to pip install -e ., and add a step that runs python setup.py compile_catalog. It also adds setuptools>=82.0.0 to l10n/requirements-l10n.txt. These are CI/localization tooling changes.

Changed components

.github/workflows/tests.ymll10n/requirements-l10n.txt

Inspect captured patch +5 / −2

diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml
index c1e14c9..32ce2ec 100644
--- a/.github/workflows/tests.yml
+++ b/.github/workflows/tests.yml
@@ -35,8 +35,10 @@ jobs:
         run: |
           sudo apt-get install libzbar0
           python -m pip install --upgrade pip
-          pip install -r requirements.txt -r tests/requirements.txt
-          pip install .
+          pip install -r requirements.txt -r tests/requirements.txt -r l10n/requirements-l10n.txt
+          pip install -e .
+      - name: Compile translations
+        run: python setup.py compile_catalog
       - name: Test with pytest
         run: |
           mkdir artifacts
diff --git a/l10n/requirements-l10n.txt b/l10n/requirements-l10n.txt
index b83c7a0..d782151 100644
--- a/l10n/requirements-l10n.txt
+++ b/l10n/requirements-l10n.txt
@@ -1 +1,2 @@
 Babel==2.16.0
+setuptools>=82.0.0

Risk score

Why this scored 15/100

Our methodology →

Potential impact 0/30

Exploitability 0/25

Stealth signal 0/15

Affected reach 0/15

Confidence 10/10

Evidence quality 5/5

Human-validated context

Community notes

Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.

No validated notes yet.

The AI analysis stands alone for now. Submit a note if you can add evidence or important context.

What the developer wrote

What changed, and why it matters

Security signals we found

Evidence from the diff

Changed components

Why this scored 15/100

Community notes

Related watches

Merge pull request #722 from Chaitanya-Keyal/psbt-huge-fee-warning

Settle is_high_fee once in parse()

Merge pull request #1032 from kdmukai/psbt_output_ownership