Skip to content

ci: enable SAST scanning with CodeQL#226

Merged
crazy-max merged 1 commit intodocker:masterfrom
crazy-max:codeql
Mar 25, 2026
Merged

ci: enable SAST scanning with CodeQL#226
crazy-max merged 1 commit intodocker:masterfrom
crazy-max:codeql

Conversation

@crazy-max
Copy link
Member

Signed-off-by: CrazyMax <1951866+crazy-max@users.noreply.github.com>
steps:
-
name: Checkout
uses: actions/checkout@v6
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you pin these by digest, or want to do that for all of them in a follow-up?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes will do all of them as follow-up with zizmor

Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Comment on lines +5 to +7
branches:
- 'master'
- 'releases/v*'
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Was this to prevent running on dependabot branches? Wondering if we should drop the branch filter overall (also helps if we would decide to do s/master/main/)

@crazy-max crazy-max merged commit 49394bd into docker:master Mar 25, 2026
135 of 139 checks passed
@crazy-max crazy-max deleted the codeql branch March 25, 2026 12:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants