Merge pull request #762 from mozilla/anca/cm-cleanup

Anca/ L10n - Add some missing sites from regions

Author: soncuteanca

choose_l10n_beta_split.py

@@ -26,7 +26,10 @@ def distribute_mappings_evenly(mappings):
         loads[min_idx] += len(regions)
     for i in range(len(balanced_splits)):
         logging.info(f"Beta split: {i}")
-        current_split = [f"{site} {' '.join(regions)}" for site, regions in balanced_splits[i].items()]
+        current_split = [
+            f"{site} {' '.join(regions)}"
+            for site, regions in balanced_splits[i].items()
+        ]
         logging.info(f"{'\n'.join(current_split)}")
     return balanced_splits
 

choose_l10n_ci_set.py

@@ -31,8 +31,19 @@ def valid_l10n_mappings():
                 if region_file.get("sites"):
                     for site in region_file.get("sites"):
                         report_mapping = True
-                        if site != "demo" and os.path.exists(f"./l10n_CM/constants/{site}/{region}"):
-                            with open(f"./l10n_CM/constants/{site}/{region}/{site}_ad.json", "r+") as ad_file, open(f"./l10n_CM/constants/{site}/{region}/{site}_cc.json", "r+") as cc_file:
+                        if site != "demo" and os.path.exists(
+                            f"./l10n_CM/constants/{site}/{region}"
+                        ):
+                            with (
+                                open(
+                                    f"./l10n_CM/constants/{site}/{region}/{site}_ad.json",
+                                    "r+",
+                                ) as ad_file,
+                                open(
+                                    f"./l10n_CM/constants/{site}/{region}/{site}_cc.json",
+                                    "r+",
+                                ) as cc_file,
+                            ):
                                 ad = json.load(ad_file)
                                 cc = json.load(cc_file)
                                 if ad.get("skip") and cc.get("skip"):

l10n_CM/region/AT.json

@@ -2,10 +2,10 @@
   "region": "AT",
   "sites": [
     "demo",
-    "giesswein",
-    "holzkern",
     "ebay",
+    "giesswein",
     "hofer",
+    "holzkern",
     "mediamarkt"
   ],
   "tests": [

l10n_CM/region/BE.json

@@ -1,12 +1,12 @@
 {
   "region": "BE",
   "sites": [
+    "brico",
     "demo",
-    "torfs",
-    "standaardboekhandel",
     "douglas",
-    "brico",
-    "toychamp"
+    "standaardboekhandel",
+    "toychamp",
+    "torfs"
   ],
   "tests": [
   ]

l10n_CM/region/CA.json

@@ -2,15 +2,17 @@
   "region": "CA",
   "sites": [
     "amazon",
-    "walmart",
+    "burtsbees",
+    "calvinklein",
+    "canadatire",
     "demo",
+    "ebay",
     "etsy",
+    "gapcanada",
+    "movieposters",
     "newbalance",
-    "canadatire",
-    "ebay",
-    "burtsbees",
     "staples",
-    "gapcanada"
+    "walmart"
   ],
   "tests": [
   ]

l10n_CM/region/DE.json

@@ -2,10 +2,14 @@
   "region": "DE",
   "sites": [
     "amazon",
-    "mediamarkt",
+    "bijoubrigitte",
     "demo",
+    "ebay",
     "justspices",
-    "bijoubrigitte"
+    "mediamarkt",
+    "newbalance",
+    "thomann",
+    "zalando"
   ],
   "tests": [
   ]

l10n_CM/region/FR.json

@@ -9,4 +9,4 @@
   ],
   "tests": [
   ]
-}
+}

l10n_CM/region/GB.json

@@ -1,12 +1,12 @@
 {
   "region": "GB",
   "sites": [
-    "ebay",
+    "assos",
     "demo",
-    "peacocks",
-    "whittard",
     "diy",
-    "assos"
+    "ebay",
+    "peacocks",
+    "whittard"
   ],
   "tests": [
   ]

l10n_CM/region/IT.json

@@ -1,12 +1,12 @@
 {
   "region": "IT",
   "sites": [
-    "ebay",
     "calvinklein",
-    "tiffany",
-    "euronics",
     "douglas",
+    "ebay",
+    "euronics",
     "thenorthface",
+    "tiffany",
     "zalando"
   ],
   "tests": [

l10n_CM/region/PL.json

@@ -1,12 +1,12 @@
 {
   "region": "PL",
   "sites": [
+    "ceneo",
+    "cocolita",
     "demo",
-    "zalando",
     "duka",
-    "ceneo",
     "tanie-leczenie",
-    "cocolita"
+    "zalando"
   ],
   "tests": [
   ]