Skip to content

Commit

Permalink
***Possible fix for sarif format execution in build pipeline
Browse files Browse the repository at this point in the history
  • Loading branch information
grolu committed Jan 17, 2025
1 parent 60e6e8a commit d674e9c
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 4 deletions.
4 changes: 1 addition & 3 deletions .ci/pipeline_definitions
Original file line number Diff line number Diff line change
Expand Up @@ -39,8 +39,6 @@ dashboard:
steps:
verify:
image: 'node:22-alpine3.20'
vars:
SARIF: 'no'
jobs:
head-update:
traits:
Expand Down Expand Up @@ -73,5 +71,5 @@ dashboard:
steps:
verify:
vars:
SARIF: 'yes'
LINT_SARIF: 'yes'

2 changes: 1 addition & 1 deletion .ci/verify
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@ yarn constraints
yarn workspace @gardener-dashboard/charts run test

# Run linting
if [ "$SARIF" = "yes" ] ; then
if [ "${LINT_SARIF+yes}" = yes ] ; then
# supposed to be run in release jobs
make lint-sarif
else
Expand Down

0 comments on commit d674e9c

Please sign in to comment.