Merge branch 'culture/release' into culture/germany

Author: ChBrain

.github/workflows/validate.yml

@@ -586,6 +586,8 @@ jobs:
     needs: [setup, L4e-hofstede-alignment]
     if: needs.setup.outputs.any == 'true'
     runs-on: ubuntu-latest
+    env:
+      PR_CHANGED_FILES: ${{ needs.setup.outputs.files }}
     steps:
       - uses: actions/checkout@v4
       - name: Set up Python
@@ -602,6 +604,9 @@ jobs:
     needs: [setup, L4f-hofstede-derived]
     if: needs.setup.outputs.any == 'true' || needs.setup.outputs.data_changed == 'true'
     runs-on: ubuntu-latest
+    env:
+      PR_CHANGED_FILES: ${{ needs.setup.outputs.files }}
+      PR_DATA_CHANGED: ${{ needs.setup.outputs.data_changed }}
     steps:
       - uses: actions/checkout@v4
       - name: Set up Python
@@ -654,4 +659,3 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - run: echo "No regions/*.md files changed - skipping validation"
-

scripts/update_hofstede_readme.py

@@ -238,10 +238,10 @@ def compute_status(declared: int, derived: int) -> tuple[int, str]:
 
 def _status_emoji(status: str) -> str:
     return {
-        "EXCELLENT": "OK",
-        "PASS":      "OK",
-        "WARN":      "WARN",
-        "FAIL":      "FAIL",
+        "EXCELLENT": "✅",
+        "PASS":      "✅",
+        "WARN":      "⚠️",
+        "FAIL":      "❌",
     }[status]
 
 

tests/test_hofstede_derived.py

@@ -1,4 +1,5 @@
 """L4f: Derived Hofstede scores vs declared — keyword scoring across all culture files."""
+import os
 import re
 import sys
 import warnings
@@ -67,6 +68,20 @@ def _derived(country_dir: Path, language: str) -> dict[str, int]:
 
 _COUNTRIES = _country_dirs()
 
+# Narrow to PR-changed countries when CI is running on a PR. Env vars are
+# set by the L4f job in .github/workflows/validate.yml; absent on push events
+# or local runs, which keeps the full corpus in scope. A Denmark-only
+# deviation shouldn't fail a Germany PR's CI.
+_pr_changed = os.environ.get("PR_CHANGED_FILES", "").strip()
+_pr_data_changed = os.environ.get("PR_DATA_CHANGED", "").strip().lower() == "true"
+if _pr_changed and not _pr_data_changed:
+    _pr_slugs = {
+        p.split("/")[2]
+        for p in _pr_changed.split()
+        if p.startswith("regions/") and len(p.split("/")) >= 4
+    }
+    _COUNTRIES = [d for d in _COUNTRIES if d.name in _pr_slugs]
+
 
 @pytest.mark.parametrize("country_dir", _COUNTRIES, ids=[c.name for c in _COUNTRIES])
 def test_derived_within_tolerance(country_dir: Path):

tests/test_hofstede_reference.py

@@ -11,6 +11,7 @@
 from __future__ import annotations
 
 import json
+import os
 import re
 from pathlib import Path
 
@@ -70,6 +71,20 @@ def _justified_dims(decisions_text: str) -> set[str]:
 _REFERENCE = _load_reference()
 _COUNTRIES = _country_dirs()
 
+# Narrow to PR-changed countries when CI is running on a PR. Env vars are
+# set by the L4g job in .github/workflows/validate.yml; absent on push events
+# or local runs, which keeps the full corpus in scope. PR_DATA_CHANGED=true
+# (the reference table itself moved) overrides and re-checks every country.
+_pr_changed = os.environ.get("PR_CHANGED_FILES", "").strip()
+_pr_data_changed = os.environ.get("PR_DATA_CHANGED", "").strip().lower() == "true"
+if _pr_changed and not _pr_data_changed:
+    _pr_slugs = {
+        p.split("/")[2]
+        for p in _pr_changed.split()
+        if p.startswith("regions/") and len(p.split("/")) >= 4
+    }
+    _COUNTRIES = [d for d in _COUNTRIES if d.name in _pr_slugs]
+
 
 @pytest.mark.parametrize("country_dir", _COUNTRIES, ids=[c.name for c in _COUNTRIES])
 def test_declared_scores_match_reference(country_dir: Path):

tests/test_update_hofstede_readme.py

@@ -161,19 +161,19 @@ def test_status_per_dimension(self):
         declared = {"PDI": 35, "IDV": 67, "UAI": 65, "MAS": 66, "LTO": 83, "IND": 40}
         derived  = {"PDI": 33, "IDV": 75, "UAI": 66, "MAS": 60, "LTO": 87, "IND": 33}
         out = upd.render_alignment_table(declared, derived)
-        assert "| PDI | 35 | 33 | 2 | OK EXCELLENT |" in out
-        assert "| IDV | 67 | 75 | 8 | OK PASS |" in out
-        assert "| UAI | 65 | 66 | 1 | OK EXCELLENT |" in out
-        assert "| MAS | 66 | 60 | 6 | OK PASS |" in out
-        assert "| LTO | 83 | 87 | 4 | OK EXCELLENT |" in out
-        assert "| IND | 40 | 33 | 7 | OK PASS |" in out
+        assert "| PDI | 35 | 33 | 2 | ✅ EXCELLENT |" in out
+        assert "| IDV | 67 | 75 | 8 | ✅ PASS |" in out
+        assert "| UAI | 65 | 66 | 1 | ✅ EXCELLENT |" in out
+        assert "| MAS | 66 | 60 | 6 | ✅ PASS |" in out
+        assert "| LTO | 83 | 87 | 4 | ✅ EXCELLENT |" in out
+        assert "| IND | 40 | 33 | 7 | ✅ PASS |" in out
 
     def test_warn_and_fail_surface(self):
         declared = {"PDI": 50, "IDV": 50}
         derived  = {"PDI": 65, "IDV": 90}
         out = upd.render_alignment_table(declared, derived)
-        assert "WARN WARN" in out  # 50 vs 65 -> gap 15 -> WARN
-        assert "FAIL FAIL" in out  # 50 vs 90 -> gap 40 -> FAIL
+        assert "⚠️ WARN" in out  # 50 vs 65 -> gap 15 -> WARN
+        assert "❌ FAIL" in out  # 50 vs 90 -> gap 40 -> FAIL
 
     def test_missing_derived_marks_na(self):
         declared = {"PDI": 35}
@@ -235,7 +235,7 @@ def test_replaces_both_tables(self):
         assert "**High** - DRIFTED" not in new_text
         assert "| Individualism (IDV) | 67 | **Moderate**" in new_text
         # Alignment table updated with new derived value
-        assert "| PDI | 35 | 33 | 2 | OK EXCELLENT |" in new_text
+        assert "| PDI | 35 | 33 | 2 | ✅ EXCELLENT |" in new_text
         # OLD EXCELLENT label removed (was wrong format)
         assert "OLD EXCELLENT" not in new_text
 
@@ -279,7 +279,7 @@ def test_alignment_only_when_dimensions_missing(self):
             text, "X", {"PDI": 35}, {"PDI": 33},
         )
         # Alignment table updated
-        assert "| PDI | 35 | 33 | 2 | OK EXCELLENT |" in new_text
+        assert "| PDI | 35 | 33 | 2 | ✅ EXCELLENT |" in new_text
         # Cultural Dimensions heading missing -> one warning
         assert len(warnings) == 1
         assert "Cultural Dimensions" in warnings[0]