-
Notifications
You must be signed in to change notification settings - Fork 46
Update golangci/golangci-lint-action digest to ba0d7d2 #954
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
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 |
|---|---|---|
|
|
@@ -25,7 +25,7 @@ jobs: | |
| echo "version=$(grep 'GOLANGCI_LINT_VERSION ?=' Makefile | sed -e 's:GOLANGCI_LINT_VERSION ?= \(.*\):\1:' -)" \ | ||
| >> $GITHUB_OUTPUT | ||
| - name: Lint with golang-ci | ||
| uses: golangci/golangci-lint-action@82606bf257cbaff209d206a39f5134f0cfbfd2ee # v9 | ||
| uses: golangci/golangci-lint-action@ba0d7d2ec06a0ea1cb5fa41b2e4a3ab91d21278a # v9 | ||
|
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. [high] protected-path This file is under the Suggested fix: Ensure a human reviewer explicitly approves this change. |
||
| with: | ||
| version: ${{ steps.golangci_version.outputs.version }} | ||
| args: "--timeout=10m --build-tags='normal periodic' --output.sarif.path=out/go-lint.sarif" | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[high] protected-path
This PR modifies a file under the
.github/protected path. The PR has no linked issue providing authorization for changes to governance/infrastructure files. While the change itself is a routine Renovate digest update for golangci/golangci-lint-action, protected-path changes always require human review and approval.Suggested fix: A maintainer should verify the digest update is legitimate and approve. No code changes are needed β this finding ensures human oversight of CI infrastructure changes.