fix(ci): correct duplicate step name in test-go.yaml#3202
fix(ci): correct duplicate step name in test-go.yaml#3202puwun wants to merge 1 commit intokubeflow:masterfrom
test-go.yaml#3202Conversation
Signed-off-by: Pavan More <pavansmore05@gmail.com>
|
🎉 Welcome to the Kubeflow Trainer! 🎉 Thanks for opening your first PR! We're happy to have you as part of our community 🚀 Here's what happens next:
Join the community:
Feel free to ask questions in the comments if you need any help or clarification! |
There was a problem hiding this comment.
Pull request overview
This PR cleans up the Go CI workflow by correcting a misleading duplicate step name so the job output accurately reflects what is being executed.
Changes:
- Renames the duplicate
Check go fmtstep toCheck go vetin.github/workflows/test-go.yamlto match themake vetcommand it runs.
Pull Request Test Coverage Report for Build 21998263416Details
💛 - Coveralls |
andreyvelich
left a comment
There was a problem hiding this comment.
Thanks @puwun!
/lgtm
/approve
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: andreyvelich The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
What this PR does / why we need it:
Renamed the duplicate
Check go fmtstep toCheck go vetintest-go.yaml, since it runs make vetChecklist: