Skip to content

Commit f04b779

Browse files
authored
Merge pull request #54 from advanced-security/GeekMasher-patch-1
feat: Update codeql-dynamic docs and on events
2 parents 28cdc42 + e458625 commit f04b779

File tree

1 file changed

+7
-1
lines changed

1 file changed

+7
-1
lines changed

.github/workflows/codeql-dynamic.yml

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,18 @@
11
name: "CodeQL Analysis"
22

33
on:
4+
# Makes the workflow reusable / callable from other workflows
45
workflow_call:
6+
workflow_dispatch:
57

68
jobs:
79
create-matrix:
810
runs-on: ubuntu-latest
911
outputs:
1012
matrix: ${{ steps.set-matrix.outputs.languages }}
1113
steps:
14+
# This step gets the languages for the repository and creates a supported
15+
# CodeQL language list that gets passed into the analyze step
1216
- name: Get languages from repo
1317
id: set-matrix
1418
uses: advanced-security/set-codeql-language-matrix@v1
@@ -18,6 +22,7 @@ jobs:
1822

1923
analyze:
2024
needs: create-matrix
25+
# Check if there are no CodeQL supported languages
2126
if: ${{ needs.create-matrix.outputs.matrix != '[]' }}
2227
name: Analyze
2328
runs-on: ubuntu-latest
@@ -28,7 +33,8 @@ jobs:
2833

2934
strategy:
3035
fail-fast: false
31-
matrix:
36+
matrix:
37+
# Create a matrix build for all supported languages
3238
language: ${{ fromJSON(needs.create-matrix.outputs.matrix) }}
3339

3440
steps:

0 commit comments

Comments
 (0)