From 4ed72df2614f641d5979cf73cf1acb223b04a531 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?D=C3=A1vid=20El-Saig?= <david.el-saig@lombiq.com>
Date: Sun, 10 Jul 2022 19:57:13 +0200
Subject: [PATCH] Remove PS analyzer invocation.

---
 .github/actions/build-dotnet/action.yml           | 15 ---------------
 .github/workflows/build-and-test-orchard-core.yml |  8 --------
 .github/workflows/build-dotnet.yml                |  8 --------
 3 files changed, 31 deletions(-)

diff --git a/.github/actions/build-dotnet/action.yml b/.github/actions/build-dotnet/action.yml
index 40d1acfbf..97e7cfbd6 100644
--- a/.github/actions/build-dotnet/action.yml
+++ b/.github/actions/build-dotnet/action.yml
@@ -50,12 +50,6 @@ inputs:
       can be omitted. Otherwise make sure to specify it to avoid an "MSB1008: Only one project can be specified." error
       that is thrown when the `*.sln` wildcard results in multiple files. The value is given to PowerShell's
       `Get-ChildItem` cmdlet as-is, so grepping is still possible but a name with spaces must be escaped separately.
-  powershell-analyzer-path:
-    required: false
-    default: "./tools/Lombiq.Analyzers.PowerShell/Invoke-Analyzer.ps1"
-    description: >
-      The path of the "Invoke-Analyzer.ps1" file to be used for linting PowerShell script files. If not found, nothing
-      happens. So if you have no use for it, feel free to ignore.
 
 runs:
   using: "composite"
@@ -105,15 +99,6 @@ runs:
         key: ${{ steps.detect-cache-configuration.outputs.key }}
         restore-keys: ${{ steps.detect-cache-configuration.outputs.restore-keys }}
 
-    - name: PowerShell Static Code Analysis
-      shell: pwsh
-      working-directory: ${{ inputs.directory }}
-      run: |
-        if (Test-Path '${{ inputs.powershell-analyzer-path }}')
-        {
-            pwsh '${{ inputs.powershell-analyzer-path }}' -ForGitHubActions
-        }
-
     - name: Build and Static Code Analysis
       shell: pwsh
       working-directory: ${{ inputs.directory }}
diff --git a/.github/workflows/build-and-test-orchard-core.yml b/.github/workflows/build-and-test-orchard-core.yml
index 8a1beab31..639a3b32e 100644
--- a/.github/workflows/build-and-test-orchard-core.yml
+++ b/.github/workflows/build-and-test-orchard-core.yml
@@ -46,13 +46,6 @@ on:
         description: >
           If set to "true", the NPM packages will be cached by GitHub. This is faster if you have a lot of NPM packages,
           but negligible or even slower if you only have a handful.
-      build-powershell-analyzer-path:
-        required: false
-        type: string
-        default: "./tools/Lombiq.Analyzers.PowerShell/Invoke-Analyzer.ps1"
-        description: >
-          The path of the "Invoke-Analyzer.ps1" file to be used for linting PowerShell script files. If not found, 
-          nothing happens. So if you have no use for it, feel free to ignore.
       build-cache-version:
         required: false
         type: string
@@ -115,7 +108,6 @@ jobs:
         enable-nuget-caching: ${{ inputs.build-enable-nuget-caching }}
         enable-npm-caching: ${{ inputs.build-enable-npm-caching }}
         cache-version: ${{ inputs.build-cache-version }}
-        powershell-analyzer-path: ${{ inputs.build-powershell-analyzer-path }}
 
     - name: Set up SQL Server
       uses: Lombiq/GitHub-Actions/.github/actions/setup-sql-server@dev
diff --git a/.github/workflows/build-dotnet.yml b/.github/workflows/build-dotnet.yml
index 4d6106dde..b1f9de3ad 100644
--- a/.github/workflows/build-dotnet.yml
+++ b/.github/workflows/build-dotnet.yml
@@ -54,13 +54,6 @@ on:
           Change this to any other value to alter the cache key, effectively invalidating the latest cache. This is the 
           only current way to force "clear" the cache (https://github.community/t/how-to-clear-cache-in-github-actions/129038/5)
           until the associated issue (https://github.com/actions/cache/issues/2) is resolved.
-      build-powershell-analyzer-path:
-        required: false
-        type: string
-        default: "./tools/Lombiq.Analyzers.PowerShell/Invoke-Analyzer.ps1"
-        description: >
-          The path of the "Invoke-Analyzer.ps1" file to be used for linting PowerShell script files. If not found, 
-          nothing happens. So if you have no use for it, feel free to ignore.
       timeout-minutes:
         required: false
         type: number
@@ -101,4 +94,3 @@ jobs:
         enable-nuget-caching: ${{ inputs.build-enable-nuget-caching }}
         enable-npm-caching: ${{ inputs.build-enable-npm-caching }}
         cache-version: ${{ inputs.build-cache-version }}
-        powershell-analyzer-path: ${{ inputs.build-powershell-analyzer-path }}