diff options
| author | Zach White <skullydazed@gmail.com> | 2021-06-24 20:48:53 -0700 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-06-24 20:48:53 -0700 |
| commit | b908275354ba6cd9dd4d393dbbedfbd2ad0f316d (patch) | |
| tree | 7efbb70c51fb917d203b629feb9eadf169f71c91 /lib/python/qmk/info.py | |
| parent | 9d4412cb8bf9512952118b190fedea72e33d28cb (diff) | |
| download | qmk_firmware-b908275354ba6cd9dd4d393dbbedfbd2ad0f316d.tar.gz qmk_firmware-b908275354ba6cd9dd4d393dbbedfbd2ad0f316d.zip | |
Optimize our jsonschema by using refs (#13271)
* fix some broken info.json files
* optimize our jsonschema using refs
* fix formatting after vscode broke it
* make flake8 happy
* cleanup
* make our schema validation more compact and flexible
Diffstat (limited to 'lib/python/qmk/info.py')
| -rw-r--r-- | lib/python/qmk/info.py | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/lib/python/qmk/info.py b/lib/python/qmk/info.py index a70bd4c31..5525f0fe6 100644 --- a/lib/python/qmk/info.py +++ b/lib/python/qmk/info.py | |||
| @@ -9,7 +9,7 @@ from milc import cli | |||
| 9 | 9 | ||
| 10 | from qmk.constants import CHIBIOS_PROCESSORS, LUFA_PROCESSORS, VUSB_PROCESSORS | 10 | from qmk.constants import CHIBIOS_PROCESSORS, LUFA_PROCESSORS, VUSB_PROCESSORS |
| 11 | from qmk.c_parse import find_layouts | 11 | from qmk.c_parse import find_layouts |
| 12 | from qmk.json_schema import deep_update, json_load, keyboard_validate, keyboard_api_validate | 12 | from qmk.json_schema import deep_update, json_load, validate |
| 13 | from qmk.keyboard import config_h, rules_mk | 13 | from qmk.keyboard import config_h, rules_mk |
| 14 | from qmk.keymap import list_keymaps | 14 | from qmk.keymap import list_keymaps |
| 15 | from qmk.makefile import parse_rules_mk_file | 15 | from qmk.makefile import parse_rules_mk_file |
| @@ -66,7 +66,7 @@ def info_json(keyboard): | |||
| 66 | 66 | ||
| 67 | # Validate against the jsonschema | 67 | # Validate against the jsonschema |
| 68 | try: | 68 | try: |
| 69 | keyboard_api_validate(info_data) | 69 | validate(info_data, 'qmk.api.keyboard.v1') |
| 70 | 70 | ||
| 71 | except jsonschema.ValidationError as e: | 71 | except jsonschema.ValidationError as e: |
| 72 | json_path = '.'.join([str(p) for p in e.absolute_path]) | 72 | json_path = '.'.join([str(p) for p in e.absolute_path]) |
| @@ -490,7 +490,7 @@ def merge_info_jsons(keyboard, info_data): | |||
| 490 | continue | 490 | continue |
| 491 | 491 | ||
| 492 | try: | 492 | try: |
| 493 | keyboard_validate(new_info_data) | 493 | validate(new_info_data, 'qmk.keyboard.v1') |
| 494 | except jsonschema.ValidationError as e: | 494 | except jsonschema.ValidationError as e: |
| 495 | json_path = '.'.join([str(p) for p in e.absolute_path]) | 495 | json_path = '.'.join([str(p) for p in e.absolute_path]) |
| 496 | cli.log.error('Not including data from file: %s', info_file) | 496 | cli.log.error('Not including data from file: %s', info_file) |
