CMP-4464: Run go fmt across the codebase - #1292
Conversation
rhmdnd
changed the title
|
@rhmdnd: This pull request references CMP-4453 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "5.0.0" version, but no target version was set. DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
|
🤖 To deploy this PR, run the following command: |
Wouldn't be nice if we can have a hook (fmt) in Claude for post changes but before commit/pushing? Or would you prefer to do such changes separately once in a while? |
Yes - absolutely. We could just check if the git tree is clean after running I think we can do that in a follow on though. |
rhmdnd
changed the title
|
@rhmdnd: This pull request references CMP-4464 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the task to target the "5.0.0" version, but no target version was set. DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
Several files had accumulated whitespace and formatting drift. This commit applies `go fmt ./...` to bring them back in line. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
3c9be62 to
8658c0e
Compare
|
🤖 To deploy this PR, run the following command: |
|
After this change lands we can merge #1300 to prevent drift in the future. |
|
/test e2e-aws-parallel |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: rhmdnd, taimurhafeez 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 |
3 participants
Several files had accumulated whitespace and formatting drift.
This commit applies
go fmt ./...to bring them back in line.Co-Authored-By: Claude Opus 4.6 noreply@anthropic.com