Make optionals explicit.

ThePalaceProject · Dec 8, 2023 · fbfd67f · fbfd67f
1 parent dbee3bb
commit fbfd67f
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/core/cli.py b/core/cli.py
@@ -1,4 +1,5 @@
 import logging
+from typing import Optional
 
 import boto3
 import click
@@ -220,7 +221,7 @@ def import_and_publish(
     group_name: str,
     result_bucket: str,
     result_key: str,
-    output_json: str = None,
+    output_json: Optional[str] = None,
 ):
 
     log.info(f"import_and_publish")

diff --git a/core/operation/publish_dashboard_from_template.py b/core/operation/publish_dashboard_from_template.py
@@ -1,5 +1,6 @@
 import json
 import time
+from typing import Optional
 
 from core.operation.baseoperation import BaseOperation
 
@@ -17,8 +18,8 @@ def __init__(
         result_bucket: str,
         result_key: str,
         s3_client,
-        dashboard_alias: str = None,
-        output_json: str = None,
+        dashboard_alias: Optional[str] = None,
+        output_json: Optional[str] = None,
         *args,
         **kwargs,
     ):