Merge pull request #43 from snyk-labs/fix/missing-required-arguments

fix: missing required arguments in apply_tags_to_*

Author: gcmurphy-snyk

pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "snyk-tags"
-version = "2.5.2"
+version = "2.5.3"
 description = "CLI tool designed to manage tags and attributes at scale"
 authors = ["EricFernandezSnyk <eric.fernandez@snyk.io>"]
 license = "MIT"

snyk_tags/attribute.py

@@ -7,7 +7,7 @@
 import typer
 from rich import print
 from snyk import SnykClient
-from typing import Dict
+from typing import Dict, Any
 
 from snyk_tags import __app_name__, __version__
 
@@ -96,8 +96,8 @@ def apply_attributes_to_projects(
     criticality: list,
     environment: list,
     lifecycle: list,
-    tenant: str,
-    filters: Dict[str, any],
+    tenant: str = "",
+    filters: Dict[str, Any] = {},
 ) -> None:
     with create_client(token=token, tenant=tenant) as client:
         for org_id in org_ids:

snyk_tags/collection.py

@@ -6,7 +6,7 @@
 import typer
 from rich import print
 from snyk import SnykClient
-from typing import Dict
+from typing import Dict, Any
 
 from snyk_tags import __app_name__, __version__, attribute, github
 
@@ -75,8 +75,8 @@ def apply_tags_to_projects(
     name: str,
     tag: str,
     key: str,
-    tenant: str,
-    filters: Dict[str, any],
+    tenant: str = "",
+    filters: Dict[str, Any] = {},
 ) -> None:
     with create_client(token=token, tenant=tenant) as client:
         for org_id in org_ids:
@@ -149,7 +149,7 @@ def tag(
     ),
     tenant: str = typer.Option(
         "",  # Default value of comamand
-        help=f"Defaults to US tenant (app.snyk.io), add 'eu', 'au' or 'us' to use alternative regional tenant. Use --tenant to change tenant.",
+        help="Defaults to US tenant (app.snyk.io), add 'eu', 'au' or 'us' to use alternative regional tenant. Use --tenant to change tenant.",
     ),
     tagKey: str = typer.Option(
         ..., help="Tag key: identifier of the tag"  # Default value of comamand
@@ -163,7 +163,7 @@ def tag(
         bold=True,
         fg=typer.colors.MAGENTA,
     )
-    apply_tags_to_projects(snyktkn, [org_id], target, tagValue, tagKey, tenant=tenant)
+    apply_tags_to_projects(snyktkn, [org_id], target, tagValue, tagKey, tenant)
 
 
 # Collection command to apply the attributes to the collection
@@ -194,7 +194,7 @@ def attributes(
     ),
     tenant: str = typer.Option(
         "",  # Default value of comamand
-        help=f"Defaults to US tenant (app.snyk.io), add 'eu', 'au' or 'us' to use alternative regional tenant. Use --tenant to change tenant.",
+        help="Defaults to US tenant (app.snyk.io), add 'eu', 'au' or 'us' to use alternative regional tenant. Use --tenant to change tenant.",
     ),
 ):
     typer.secho(
@@ -209,5 +209,5 @@ def attributes(
         [criticality],
         [environment],
         [lifecycle],
-        tenant=tenant,
+        tenant,
     )