Fix a sloppy bug in the sanity checks in CI.

This was sloppy -- and was never running :/ because it was mis-indented,
causing us to not actually check any schemas for validity.

This is part of why for loops in tests are so bad, but we are a bit
loose in this file (particularly because I've avoided adding
dependencies so it's slightly easier to run for those who aren't Python
developers). We should possibly refactor a bit anyhow so that an error
is raised if the for loops are empty.

Author: Julian

bin/jsonschema_suite

@@ -97,12 +97,12 @@ def versions_and_validators():
         if not version.is_dir():
             continue
 
-            Validator = VALIDATORS.get(version.name)
-            if Validator is None:
-                warnings.warn(f"No schema validator for {version.name}")
-                continue
+        Validator = VALIDATORS.get(version.name)
+        if Validator is None:
+            warnings.warn(f"No schema validator for {version.name}")
+            continue
 
-            yield version, Validator
+        yield version, Validator
 
 
 class SanityTests(unittest.TestCase):