Merge pull request #674 from mozilla/philimon/l10n_remove_default

Philimon/l10n_remove_default_mapping

Author: philimon-reset

.github/workflows/l10n.yml

@@ -109,6 +109,7 @@ jobs:
             echo "Running tests for: $line";
             pipenv run python -m l10n_CM.run_l10n --fx-executable="$FX_EXECUTABLE" $line || SCRIPT_EXIT_CODE=$?;
           done < selected_l10n_mappings;
+          ls artifacts-mac/* &>/dev/null || echo "No mappings to report" >> artifacts-mac/placeholder.txt
           mv -n artifacts/* artifacts-mac/ || true;
           EXIT_CODE=$(cat TEST_EXIT_CODE);
           if [ $EXIT_CODE != 0 ]; then
@@ -191,6 +192,7 @@ jobs:
             echo "Running tests for: $line";
             DISPLAY=:99 pipenv run python -m l10n_CM.run_l10n --fx-executable="$FX_EXECUTABLE" $line || SCRIPT_EXIT_CODE=$?;
           done < selected_l10n_mappings;
+          ls artifacts-linux/* &>/dev/null || echo "No mappings to report" >> artifacts-linux/placeholder.txt
           mv -n artifacts/* artifacts-linux/ || true;
           EXIT_CODE=$(cat TEST_EXIT_CODE);
           if [ $EXIT_CODE != 0 ]; then
@@ -317,8 +319,8 @@ jobs:
                   $SCRIPT_EXIT_CODE = $_.Exception.HResult
               }
           }
-          rm artifacts/assets -r -Force
-          Get-ChildItem -Path "artifacts" | ForEach-Object {
+          rm artifacts/assets -r -Force -ErrorAction SilentlyContinue
+          Get-ChildItem -Path "artifacts" -ErrorAction SilentlyContinue | ForEach-Object {
               $destPath = "artifacts-win\" + $_.Name
               if (-not (Test-Path -Path $destPath)) {
                   try {
@@ -328,6 +330,10 @@ jobs:
                   }
               }
           }
+          mkdir artifacts-win -Force
+          if (-not (Test-Path -Path "artifacts-win\*")) {
+            New-Item -Path "artifacts-win\placeholder.txt" -ItemType File -Value "no mappings to report."
+          }
           $EXIT_CODE = Get-Content -Path "TEST_EXIT_CODE"
           if ($EXIT_CODE -ne 0) {
               exit [int]$EXIT_CODE
@@ -363,7 +369,7 @@ jobs:
         with:
           name: artifacts-win
           path: artifacts-win
-      - name: Download Windows artifact
+      - name: Download Linux artifact
         uses: actions/download-artifact@v4
         with:
           name: artifacts-linux

choose_l10n_ci_set.py

@@ -135,8 +135,6 @@ def process_changed_file(f, selected_mappings):
 
     # Run all the tests for all mappings if any core l10n model, component, conftest, or tests are changed.
     selected_mappings = defaultdict(set)
-    # by default, have the demo page run for US
-    selected_mappings["demo"] = {"US"}
     for f in committed_files:
         for re_val in re_set_all:
             if re_val.match(f):