Merge pull request #239 from oke-py/fix/disable-windows-jobs

fix: disable scan-on-windows job and add PowerShell execution policy setting
This commit is contained in:
Naoki Oketani
2025-05-04 12:34:41 +09:00
committed by GitHub
2 changed files with 29 additions and 18 deletions

View File

@@ -23,21 +23,28 @@ jobs:
dedupe_issues: true
production_flag: true
scan-on-windows:
name: npm audit
runs-on: windows-latest
steps:
- uses: actions/checkout@v4
# https://stackoverflow.com/questions/72401421/message-npm-warn-config-global-global-local-are-deprecated-use-loc
- run: |
npm install npm-windows-upgrade --location=global
npm-windows-upgrade --npm-version latest
- name: install dependencies
run: npm ci
- uses: oke-py/npm-audit-action@v3
with:
github_token: ${{ secrets.GITHUB_TOKEN }}
issue_assignees: oke-py
issue_labels: vulnerability
dedupe_issues: true
production_flag: true
# scan-on-windows job disabled due to npm.cmd EINVAL error on Windows
# scan-on-windows:
# name: npm audit
# runs-on: windows-latest
# steps:
# - uses: actions/checkout@v4
# # Set PowerShell execution policy for this process
# - name: Set PowerShell execution policy
# shell: powershell
# run: Set-ExecutionPolicy -ExecutionPolicy RemoteSigned -Scope Process -Force
# # https://stackoverflow.com/questions/72401421/message-npm-warn-config-global-global-local-are-deprecated-use-loc
# - name: Upgrade npm
# shell: powershell
# run: |
# npm install npm-windows-upgrade --location=global
# npm-windows-upgrade --npm-version latest --no-prompt
# - name: install dependencies
# run: npm ci
# - uses: oke-py/npm-audit-action@v3
# with:
# github_token: ${{ secrets.GITHUB_TOKEN }}
# issue_assignees: oke-py
# issue_labels: vulnerability
# dedupe_issues: true
# production_flag: true

View File

@@ -84,6 +84,10 @@ jobs:
# - uses: actions/setup-node@v4
# with:
# node-version: ${{ matrix.node }}
# # Set PowerShell execution policy for this process
# - name: Set PowerShell execution policy
# shell: powershell
# run: Set-ExecutionPolicy -ExecutionPolicy RemoteSigned -Scope Process -Force
# # Node.js 20 already includes a recent npm version, so npm upgrade is not needed
# - name: Run npm audit action
# uses: ./