Skip to content

Commit

Permalink
Remove PS analyzer invocation.
Browse files Browse the repository at this point in the history
  • Loading branch information
sarahelsaig committed Jul 10, 2022
1 parent b14e882 commit 4ed72df
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 31 deletions.
15 changes: 0 additions & 15 deletions .github/actions/build-dotnet/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -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"
Expand Down Expand Up @@ -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 }}
Expand Down
8 changes: 0 additions & 8 deletions .github/workflows/build-and-test-orchard-core.yml
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down Expand Up @@ -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
Expand Down
8 changes: 0 additions & 8 deletions .github/workflows/build-dotnet.yml
Original file line number Diff line number Diff line change
Expand Up @@ -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.saobby.my.eu.orgmunity/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
Expand Down Expand Up @@ -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 }}

0 comments on commit 4ed72df

Please sign in to comment.