-
-
Notifications
You must be signed in to change notification settings - Fork 581
ci: Integrate zizmor checks to repo #818
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
0b07b03
9bd1771
fbc5b29
c23c9db
af64d38
4700970
3116c7d
b62d907
e50d2e0
e2ba9e7
3b0b13b
2e36691
dea6ba1
6186781
63d6d97
a9591da
d8a7602
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,7 +1,7 @@ | ||
| name: Validate PR title | ||
|
|
||
| on: | ||
| pull_request_target: | ||
| pull_request: | ||
| types: | ||
| - opened | ||
| - edited | ||
|
|
||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @MaxymVlasov FYI, I got a reusable workflow that integrates with GitHub's security reporting interfaces natively: https://github.com/zizmorcore/workflow/blob/3bb5e95068d0f44b6d2f3f7e91379bed1d2f96a8/.github/workflows/reusable-zizmor.yml. If you add it, you'll get inline alerts right inside PRs. |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -47,6 +47,12 @@ repos: | |
| hooks: | ||
| - id: gitleaks | ||
|
|
||
| # Github Action static analysis tool | ||
| - repo: https://github.com/woodruffw/zizmor-pre-commit | ||
| rev: v1.9.0 | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Tag and not commit hash? I thought you tend to trust commits rather than tags 🤔
Collaborator
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. We need to switch out from Renovate to pre-commit-freeze then. I'll do it as separate PR
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I think native |
||
| hooks: | ||
| - id: zizmor | ||
|
|
||
| # Dockerfile | ||
| - repo: https://github.com/hadolint/hadolint | ||
| rev: v2.13.1-beta | ||
|
|
||
Uh oh!
There was an error while loading. Please reload this page.