PyPI - bitwarden_workflow_linter - Versions diffs - 0.1.1__tar.gz → 0.2.1__tar.gz - Mend

@@ -25,7 +25,7 @@ class RuleUnderscoreOutputs(Rule):
             A Settings object that contains any default, overridden, or custom settings
             required anywhere in the application.
         """
-        self.message = "outputs with more than one word must use an underscore"
+        self.message = "outputs with more than one word should use an underscore"
         self.on_fail = LintLevels.WARNING
         self.compatibility = [Workflow, Job, Step]
         self.settings = settings
@@ -90,14 +90,18 @@ class RuleUnderscoreOutputs(Rule):
         if isinstance(obj, Workflow):
             if obj.on.get("workflow_dispatch"):
                 if obj.on["workflow_dispatch"].get("outputs"):
-                    outputs.extend(obj.on["workflow_dispatch"]["outputs"].keys())
+                    for output, _ in obj.on["workflow_dispatch"]["outputs"].items():
+                        outputs.append(output)
             if obj.on.get("workflow_call"):
-                outputs.extend(obj.on["workflow_call"]["outputs"].keys())
+                if obj.on["workflow_call"].get("outputs"):
+                    for output, _ in obj.on["workflow_call"]["outputs"].items():
+                        outputs.append(output)
         if isinstance(obj, Job):
             if obj.outputs:
-                outputs.extend(obj.outputs.keys())
+                for output in obj.outputs.keys():
+                    outputs.append(output)
         if isinstance(obj, Step):
             if obj.run:
@@ -107,10 +111,18 @@ class RuleUnderscoreOutputs(Rule):
                     )
                 )
-        for output_name in outputs:
-            if "-" in output_name:
-                return False, (
-                    f"Hyphen found in {obj.__class__.__name__} output: {output_name}"
-                )
+        correct = True
+        offending_keys = []
+        for name in outputs:
+            if "-" in name:
+                offending_keys.append(name)
+                correct = False
+        if correct:
+            return True, ""
-        return True, ""
+        return (
+            False,
+            f"{obj.__class__.__name__} {self.message}: ({' ,'.join(offending_keys)})",
+        )

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: bitwarden_workflow_linter
-Version: 0.1.1
+Version: 0.2.1
 Summary: Custom GitHub Action Workflow Linter
 Project-URL: Homepage, https://github.com/bitwarden/workflow-linter
 Project-URL: Issues, https://github.com/bitwarden/workflow-linter/issues

@@ -1,3 +1,3 @@
 """Metadata for Workflow Linter."""
-__version__ = "0.1.1"
+__version__ = "0.2.1"

@@ -4,6 +4,7 @@ enabled_rules:
   - bitwarden_workflow_linter.rules.pinned_job_runner.RuleJobRunnerVersionPinned
   - bitwarden_workflow_linter.rules.job_environment_prefix.RuleJobEnvironmentPrefix
   - bitwarden_workflow_linter.rules.step_pinned.RuleStepUsesPinned
+  - bitwarden_workflow_linter.rules.step_pinned.RuleStepUsesPinned
   - bitwarden_workflow_linter.rules.underscore_outputs.RuleUnderscoreOutputs
 approved_actions_path: default_actions.json

@@ -54,6 +54,7 @@ class LinterCmd:
         )
         parser_lint.add_argument("-f", "--files", action="append", help="files to lint")
         parser_lint.add_argument(
+            "-o",
             "--output",
             action="store",
             help="output format: [stdout|json|md]",
@@ -137,9 +138,10 @@ class LinterCmd:
             if os.path.isfile(path):
                 workflow_files.append(path)
             elif os.path.isdir(path):
-                for subdir, _, files in os.walk(path):
-                    for filename in files:
-                        filepath = subdir + os.sep + filename
+                for dirpath, dirnames, filenames in os.walk(path):
+                    dirnames[:] = []
+                    for file in filenames:
+                        filepath = dirpath + os.sep + file
                         if filepath.endswith((".yml", ".yaml")):
                             workflow_files.append(filepath)

@@ -58,11 +58,12 @@ class RuleJobEnvironmentPrefix(Rule):
         incorrectly named environment variables.
         """
         correct = True
+        allowed_envs = {"NODE_OPTION", "NUGET_PACKAGES", "MINT_PATH", "MINT_LINK_PATH"}
         if obj.env:
             offending_keys = []
             for key in obj.env.keys():
-                if key[0] != "_":
+                if key not in allowed_envs and key[0] != "_":
                     offending_keys.append(key)
                     correct = False

@@ -41,8 +41,8 @@ class RuleStepUsesApproved(Rule):
         if "@" not in obj.uses:
             return True
-        ## Force pass for any bitwarden/gh-actions
-        if obj.uses.startswith("bitwarden/gh-actions"):
+        ## Force pass for any bitwarden/
+        if obj.uses.startswith("bitwarden/"):
             return True
         return False

@@ -82,7 +82,7 @@ class RuleStepUsesPinned(Rule):
         path, ref = obj.uses.split("@")
-        if path.startswith("bitwarden/gh-actions"):
+        if path.startswith("bitwarden/"):
             if ref == "main":
                 return True, ""
             return False, "Please pin to main"

@@ -25,7 +25,7 @@ jobs:
         uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # v4.1.1
       - name: Test Internal Action
-        uses: bitwarden/gh-actions/get-keyvault-secrets@main
+        uses: bitwarden/ios/.github/actions/dispatch-and-download@main
       - name: Test Local Action
         uses: ./actions/test-action

@@ -234,7 +234,7 @@ def test_rule_on_incorrect_step(rule, incorrect_workflow):
     result, message = rule.fn(incorrect_workflow.jobs["job-key"].steps[1])
     assert result is False
-    assert message == "Hyphen found in Step output: test-key-2"
+    assert "outputs with more than one word should use an underscore" in message
     result, _ = rule.fn(incorrect_workflow.jobs["job-key"].steps[2])
     assert result is True

bitwarden_workflow_linter 0.1.1__tar.gz → 0.2.1__tar.gz

bitwarden_workflow_linter 0.1.1tar.gz → 0.2.1tar.gz