Skip to content

Commit 6da6be4

Browse files
authored
Remove scan-required from matrix
1 parent 3fd7baf commit 6da6be4

File tree

1 file changed

+5
-6
lines changed

1 file changed

+5
-6
lines changed

.github/workflows/codeql-monorepo.yml

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -19,10 +19,10 @@ on:
1919
pull_request:
2020
branches: ["main"]
2121
types:
22-
- opened
23-
- reopened
24-
- synchronize
25-
- closed
22+
- opened # scan new PRs
23+
- reopened # scan PRs that are reopened
24+
- synchronize # scan newly pushed commits to PRs
25+
- closed # republish results from merged PRs
2626

2727
jobs:
2828
changes:
@@ -75,7 +75,7 @@ jobs:
7575
projects-json: monorepo-projects.json
7676

7777
scan:
78-
#if: needs.changes.outputs.scan-required == true
78+
if: needs.changes.outputs.scan-required == 'true'
7979
runs-on: ubuntu-latest
8080
permissions:
8181
contents: read
@@ -85,7 +85,6 @@ jobs:
8585
strategy:
8686
matrix:
8787
project: ${{ fromJson(needs.changes.outputs.projects).projects }}
88-
scan-required: ${{ steps.changes.outputs.scan-required }}
8988
steps:
9089
- name: Analyze code
9190
uses: advanced-security/monorepo-code-scanning-action/scan@main

0 commit comments

Comments
 (0)