Improve comments

vega · Dec 25, 2022 · d8b1a2a · d8b1a2a
1 parent 44d8476
commit d8b1a2a
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/altair/utils/schemapi.py b/altair/utils/schemapi.py
@@ -59,12 +59,14 @@ def validate_jsonschema(*args, **kwargs):
             # e.g. '#/definitions/ValueDefWithCondition<MarkPropFieldOrDatumDef,
             # (Gradient|string|null)>' would be a valid $ref in a Vega-Lite schema but
             # it is not a valid URI reference due to the characters such as '<'.
-            # This is fine and we can disable this format check below-
+            # This is fine and we can disable this format check below.
             for format_name in ["uri-reference"]:
                 try:
                     checker = validator_cls.FORMAT_CHECKER.checkers.pop(format_name)
                     removed_format_checkers.append((format_name, checker))
                 except KeyError:
+                    # Format checks are only set by jsonschema if it can import
+                    # the relevant dependencies
                     continue
         output = jsonschema.validate(*args, **kwargs)
     finally:

diff --git a/tools/schemapi/schemapi.py b/tools/schemapi/schemapi.py
@@ -57,12 +57,14 @@ def validate_jsonschema(*args, **kwargs):
             # e.g. '#/definitions/ValueDefWithCondition<MarkPropFieldOrDatumDef,
             # (Gradient|string|null)>' would be a valid $ref in a Vega-Lite schema but
             # it is not a valid URI reference due to the characters such as '<'.
-            # This is fine and we can disable this format check below-
+            # This is fine and we can disable this format check below.
             for format_name in ["uri-reference"]:
                 try:
                     checker = validator_cls.FORMAT_CHECKER.checkers.pop(format_name)
                     removed_format_checkers.append((format_name, checker))
                 except KeyError:
+                    # Format checks are only set by jsonschema if it can import
+                    # the relevant dependencies
                     continue
         output = jsonschema.validate(*args, **kwargs)
     finally: