🤖 ci: remove duplicate Go version input in govulncheck action#20
Conversation
|
@codex review |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 7b002a328e
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
ThomasK33
force-pushed
the
ci-github-he70
branch
from
7b002a3 to
0dee959
Compare
|
@codex review Please take another look. |
|
Codex Review: Didn't find any major issues. Delightful! ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
Use `go-version-input` for `golang/govulncheck-action` instead of `go-version-file` to avoid setup-go warnings about both version input modes being set. --- _Generated with `mux` • Model: `openai:gpt-5.3-codex` • Thinking: `xhigh` • Cost: `$0.09`_ <!-- mux-attribution: model=openai:gpt-5.3-codex thinking=xhigh costs=0.09 -->
ThomasK33
force-pushed
the
ci-github-he70
branch
from
0dee959 to
ae355c9
Compare
|
@codex review Added an inline YAML comment explaining why go.mod is parsed before govulncheck. |
|
Codex Review: Didn't find any major issues. Swish! ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
1 participant
Summary
Fix the CI
govulncheckstep to use a single Go version input and remove the duplicate-version warning in workflow annotations.Background
golang/govulncheck-actioninternally invokesactions/setup-gowithgo-version(go-version-input).Providing
go-version-fileat the same time caused setup-go to emit:"Both go-version and go-version-file inputs are specified, only go-version will be used."
Implementation
.github/workflows/ci.yamlin thelintjob'sRun govulncheckstep:go-version-file: go.modgo-version-input: 1.25.7Validation
make verify-vendormake testmake buildmake lintgo run github.com/rhysd/actionlint/cmd/actionlint@v1.7.10Risks
Low risk. The change is limited to workflow configuration for a single action input and aligns the configured Go version with
go.mod.Generated with
mux• Model:openai:gpt-5.3-codex• Thinking:xhigh• Cost:$0.09