Merge pull request #15 from TechForPalestine/add_reason

TechForPalestine · Jan 30, 2024 · f478606 · f478606
2 parents c55b46e + 9b0143c
commit f478606
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/schemas/brand_schema.yaml b/schemas/brand_schema.yaml
@@ -22,6 +22,8 @@ properties:
       enum:
         - operations_in_israel
         - operations_in_settlements
+        - executive_supports_israel
+        - hiring_discrimination
   countries:
     description: |-
       A list of countries (ISO alpha-2 country codes) that the brand operates in. 

diff --git a/scripts/validate_yaml.py b/scripts/validate_yaml.py
@@ -42,9 +42,14 @@ def main():
     company_schema = load_yaml(os.path.join(root_path, "schemas/company_schema.yaml"))
     company_files = glob.glob(os.path.join(root_path, "data/companies/") + "*.yaml")
     print("Validating", len(company_files), "companies")
+    failed = False
     for file in company_files:
         if not validate_with_schema(file, company_schema):
-            exit(1)
+            failed = True
+
+    if failed:
+        exit(1)
+
     print("All companies are valid.")