Skip to content
This repository has been archived by the owner on Nov 27, 2024. It is now read-only.

Commit

Permalink
ci: upload shellcheck SARIF as security-related (#134)
Browse files Browse the repository at this point in the history
We were uploading the shellcheck artifact as an artifact to download,
which doesn't get reflected in the security tab on github.  Instead, we
should be uploading it as a sarif file to be consumed by github.

Signed-off-by: Andy Sadler <ansadler@redhat.com>
Co-authored-by: Francesco Ilario <filario@redhat.com>
  • Loading branch information
sadlerap and filariow authored May 16, 2024
1 parent 4dfa030 commit bca5bff
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions .github/workflows/security.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -87,7 +87,6 @@ jobs:
- if: ${{ always() }}
name: Upload ShellCheck defects
uses: actions/upload-artifact@v4
uses: github/codeql-action/upload-sarif@v3
with:
name: ShellCheck SARIF
path: results.sarif
sarif_file: results.sarif

0 comments on commit bca5bff

Please sign in to comment.