Reformat

lint-workflow/src/actions.py

@@ -54,9 +54,7 @@ def extend_parser(subparsers: argparse.ArgumentParser) -> argparse.ArgumentParse
         subparsers_actions = parser_actions.add_subparsers(
             required=True, dest="actions_command"
         )
-        subparsers_actions.add_parser(
-            "update", help="update action versions"
-        )
+        subparsers_actions.add_parser("update", help="update action versions")
         parser_actions_add = subparsers_actions.add_parser(
             "add", help="add action to approved list"
         )
@@ -80,15 +78,15 @@ def get_github_api_response(
         if response.status == 403 and response.reason == "rate limit exceeded":
             logging.error(
                 "Failed to call GitHub API for action: %s due to rate limit exceeded.",
-                action_name
+                action_name,
             )
             return None
 
         if response.status == 401 and response.reason == "Unauthorized":
             logging.error(
                 "Failed to call GitHub API for action: %s: %s.",
                 action_name,
-                response.data
+                response.data,
             )
             return None
 

lint-workflow/src/lint.py

@@ -16,7 +16,8 @@ class LinterCmd:
     Supporting logic is supplied to:
       - build out the list of Rules desired
       - select and validate the workflow files to lint
-      """
+    """
+
     def __init__(self, settings: Settings = None) -> None:
         """Initailized the LinterCmd class.
 

lint-workflow/src/load.py

@@ -111,12 +111,13 @@ class Rules:
     workflow. It also assists in making sure the Rules that apply to multiple
     types are not skipped.
     """
+
     workflow: List[Rule] = []
     job: List[Rule] = []
     step: List[Rule] = []
 
     def __init__(self, settings: Settings) -> None:
-        """Initializes the Rules 
+        """Initializes the Rules
 
         Args:
           settings:

lint-workflow/src/models/job.py

@@ -17,6 +17,7 @@ class Job:
     Rules against. If a new Rule requies a key that is missing, the attribute should
     be added to this class to make it available for use in linting.
     """
+
     runs_on: str = field(metadata=config(field_name="runs-on"))
     key: Optional[str] = None
     name: Optional[str] = None

lint-workflow/src/models/step.py

@@ -15,6 +15,7 @@ class Step:
     Rules against. If a new Rule requies a key that is missing, the attribute should
     be added to this class to make it available for use in linting.
     """
+
     key: Optional[int] = None
     job: Optional[str] = None
     name: Optional[str] = None

lint-workflow/src/models/workflow.py

@@ -20,6 +20,7 @@ class Workflow:
     See src/models/job.py for an example if the key in the workflow data does not map
     one-to-one in the model (ex. 'with' => 'uses_with')
     """
+
     key: str = ""
     name: Optional[str] = None
     on: Optional[CommentedMap] = None

lint-workflow/src/rule.py

@@ -9,6 +9,7 @@
 
 class RuleExecutionException(Exception):
     """Exeception for the Base Rule class."""
+
     pass
 
 

lint-workflow/src/rules/job_environment_prefix.py

@@ -22,6 +22,7 @@ class RuleJobEnvironmentPrefix(Rule):
     variables are normally at the top of the file and Step level ones are pretty
     visible when debugging a shell Step.
     """
+
     def __init__(self, settings: Settings = None) -> None:
         """RuleJobEnvironmentPrefix constructor to override the Rule class.
 

lint-workflow/src/rules/name_capitalized.py

@@ -13,6 +13,7 @@ class RuleNameCapitalized(Rule):
 
     A simple standard to help keep uniformity in naming.
     """
+
     def __init__(self, settings: Settings = None) -> None:
         """Contructor for RuleNameCapitalized to override the Rule class.
 

lint-workflow/src/rules/name_exists.py

@@ -17,6 +17,7 @@ class RuleNameExists(Rule):
 
     It also helps with uniformity of runs.
     """
+
     def __init__(self, settings: Settings = None) -> None:
         """Contructor for RuleNameCapitalized to override Rule class.
 

lint-workflow/src/rules/pinned_job_runner.py

@@ -15,6 +15,7 @@ class RuleJobRunnerVersionPinned(Rule):
     in the past. To avoid this and prevent a single event from breaking the majority of
     our pipelines, we pin the versions.
     """
+
     def __init__(self, settings: Settings = None) -> None:
         """Constructor for RuleJobRunnerVersionPinned to override Rule class.
 

lint-workflow/src/rules/step_approved.py

@@ -15,6 +15,7 @@ class RuleStepUsesApproved(Rule):
     are required to pass a security review and be added to the pre-approved list to
     check against.
     """
+
     def __init__(self, settings: Settings = None) -> None:
         """Constructor for RuleStepUsesApproved to override Rule class.
 

lint-workflow/src/rules/step_pinned.py

@@ -23,6 +23,7 @@ class RuleStepUsesPinned(Rule):
     spamming a bunch of PRs across all of our repos when `bitwarden/gh-actions` is
     updated.
     """
+
     def __init__(self, settings: Settings = None) -> None:
         """Constructor for RuleStepUsesPinned to override base Rule.
 

lint-workflow/src/utils.py

@@ -100,6 +100,7 @@ class SettingsError(Exception):
 
 class Settings:
     """Class that contains configuration-as-code for any portion of the app."""
+
     enabled_rules: list[str]
     approved_actions: dict[str, Action]