🩹 [Refactor]: Rename and restructure Test-SourceCode job to Test-PSModule-SourceCode and add Invoke-ScriptAnalyzer job for improved linting

MariusStorhaug · MariusStorhaug · commit 32b6dc2ba70e · 2025-02-24T00:22:21.000+01:00
diff --git a/.github/workflows/Test-SourceCode.yml b/.github/workflows/Test-SourceCode.yml
@@ -50,34 +50,40 @@ permissions:
   contents: read # to checkout the repo and create releases on the repo
 
 jobs:
-  Test-SourceCode:
+  Test-PSModule-SourceCode:
     name: Test source code (${{ inputs.RunsOn }})
     if: ${{ !(contains(inputs.SkipTests, 'All') || contains(inputs.SkipTests, 'SourceCode') || contains(inputs.SkipTests, inputs.OS)) }}
     runs-on: ${{ inputs.RunsOn }}
     steps:
       - name: Checkout Code
         uses: actions/checkout@v4
 
-      - name: Debug
-        if: ${{ inputs.Debug }}
-        uses: PSModule/Debug@v0
-
-      - name: Initialize environment
-        uses: PSModule/Initialize-PSModule@v1
+      - name: Test source code
+        id: test
+        uses: PSModule/Test-PSModule@v3
         with:
           Debug: ${{ inputs.Debug }}
           Prerelease: ${{ inputs.Prerelease }}
           Verbose: ${{ inputs.Verbose }}
           Version: ${{ inputs.Version }}
+          Path: ${{ inputs.Path }}
+          Settings: SourceCode
 
-      - name: Test source code
+  Invoke-ScriptAnalyzer-SourceCode:
+    name: Lint source code (${{ inputs.RunsOn }})
+    if: ${{ !(contains(inputs.SkipTests, 'All') || contains(inputs.SkipTests, 'SourceCode') || contains(inputs.SkipTests, inputs.OS)) }}
+    runs-on: ${{ inputs.RunsOn }}
+    steps:
+      - name: Checkout Code
+        uses: actions/checkout@v4
+
+      - name: Lint source code
         id: test
-        uses: PSModule/Test-PSModule@v2
+        uses: PSModule/Invoke-ScriptAnalyzer@v2
         with:
-          Name: ${{ inputs.Name }}
-          Path: ${{ inputs.Path }}
-          TestType: SourceCode
           Debug: ${{ inputs.Debug }}
           Prerelease: ${{ inputs.Prerelease }}
           Verbose: ${{ inputs.Verbose }}
           Version: ${{ inputs.Version }}
+          Path: ${{ inputs.Path }}
+          Settings: SourceCode
diff --git a/.github/workflows/workflow.yml b/.github/workflows/workflow.yml
@@ -102,6 +102,7 @@ permissions:
 
 jobs:
   TestSourceCode:
+    name: Test source code
     strategy:
       matrix:
         include:
@@ -143,11 +144,11 @@ jobs:
       Path: ${{ inputs.Path }}
       ModulesOutputPath: ${{ inputs.ModulesOutputPath }}
       DocsOutputPath: ${{ inputs.DocsOutputPath }}
+      SiteOutputPath: ${{ inputs.SiteOutputPath }}
       Debug: ${{ inputs.Debug }}
       Prerelease: ${{ inputs.Prerelease }}
       Verbose: ${{ inputs.Verbose }}
       Version: ${{ inputs.Version }}
-      SiteOutputPath: ${{ inputs.SiteOutputPath }}
 
   TestModule:
     if: ${{ needs.Build-Module.result == 'success' && !cancelled() }}
