Skip to content

Commit

Permalink
Merge pull request #345 from tisutisu/add-myself-to-owners
Browse files Browse the repository at this point in the history
chore: Adjusting OWNERS file and fix Github Action CI
  • Loading branch information
tisutisu authored Sep 12, 2024
2 parents 6a3c822 + a49e045 commit e4989d2
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 4 deletions.
5 changes: 3 additions & 2 deletions .github/workflows/pr.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -27,12 +27,13 @@ jobs:

# https://github.com/securego/gosec/blob/12be14859bc7d4b956b71bef0b443694aa519d8a/README.md#integrating-with-code-scanning
- name: Run Gosec Security Scanner
uses: securego/gosec@master
# pin gosec to use v2.21.0 temporarily, once upstream issue https://github.com/securego/gosec/issues/1214 is fixed, revert it to use master
uses: securego/[email protected]
with:
# we let the report trigger content trigger a failure using the GitHub Security features.
args: '-no-fail -fmt sarif -out results.sarif ./...'
- name: Upload SARIF file
uses: github/codeql-action/upload-sarif@v2
uses: github/codeql-action/upload-sarif@v3
with:
# Path to SARIF file relative to the root of the repository
sarif_file: results.sarif
Expand Down
4 changes: 2 additions & 2 deletions OWNERS
Original file line number Diff line number Diff line change
Expand Up @@ -3,12 +3,12 @@
approvers:
- mmorhun
- psturc
- Michkov
- tisutisu

reviewers:
- mmorhun
- psturc
- Michkov
- tisutisu

# 'Build team' members that are not members of openshift github org (could be added in a future):
# - brunoapimentel
Expand Down

0 comments on commit e4989d2

Please sign in to comment.