From a78dc70a502da2e3096861975052fbb0498e2429 Mon Sep 17 00:00:00 2001 From: Archana Date: Sun, 13 Mar 2022 17:57:45 +0530 Subject: [PATCH] Fix JsonObject type for MyPy Workaround a MyPy error by marking the driver json Object as of type dict and indexable. Signed-off-by: Archana Signed-off-by: Asfandyar Orakzai --- scripts/generate_driver_wrappers.py | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/scripts/generate_driver_wrappers.py b/scripts/generate_driver_wrappers.py index 3d9be88e2..3c71202d3 100755 --- a/scripts/generate_driver_wrappers.py +++ b/scripts/generate_driver_wrappers.py @@ -30,7 +30,9 @@ import jinja2 from mbedtls_dev import build_tree JSONSchema = NewType('JSONSchema', object) -Driver = NewType('Driver', object) +# The Driver is an Object, but practically it's indexable and can called a dictionary to +# keep MyPy happy till MyPy comes with a more composite type for JsonObjects. +Driver = NewType('Driver', dict) def render(template_path: str, driver_jsoncontext: list) -> str: """ @@ -63,7 +65,6 @@ def validate_json(driverjson_data: Driver, driverschema_list: dict) -> bool: Validate the Driver JSON against an appropriate schema the schema passed could be that matching an opaque/ transparent driver. """ - driver_type = driverjson_data["type"] driver_prefix = driverjson_data["prefix"] try: