Skip file (but not remaining files) if certain failures occur

Author: homebysix

pre_commit_hooks/check_preference_manifests.py

@@ -393,12 +393,13 @@ def main(argv=None):
         except (ExpatError, ValueError) as err:
             print("{}: plist parsing error: {}".format(filename, err))
             retval = 1
+            continue  # No need to continue checking this file
 
         # Check for presence of required keys.
         required_keys = ("pfm_title", "pfm_domain", "pfm_description")
         if not validate_required_keys(manifest, required_keys, "<root dict>", filename):
             retval = 1
-            break  # No need to continue checking this file
+            continue  # No need to continue checking this file
 
         # Ensure pfm_format_version has expected value
         if manifest.get("pfm_format_version", 1) != 1: