Skip to content

[Misc] build: gofmt actions fix#208

Merged
Pavan-SAP merged 2 commits into
mainfrom
gofmt-fix
May 22, 2026
Merged

[Misc] build: gofmt actions fix#208
Pavan-SAP merged 2 commits into
mainfrom
gofmt-fix

Conversation

@Pavan-SAP
Copy link
Copy Markdown
Member

Print out result of gofmt correctly

Comment thread internal/transformer/object_transformer.go Fixed
Use latest version(s) of github actions.
The command now exits with 1 when a diff exists!
@Pavan-SAP
Copy link
Copy Markdown
Member Author

@anirudhprasad-sap : This also fixes the gofmt log issue: https://github.com/SAP/cap-operator-lifecycle/actions/runs/26253916162/job/77271818167#step:5:21
(FYI, as of end of last year i.e. go 1.26 - gofmt exits with 1 when diff is enabled - so the workaround isn't needed anymore)

@Pavan-SAP Pavan-SAP enabled auto-merge (rebase) May 22, 2026 06:53
Copy link
Copy Markdown
Contributor

@anirudhprasad-sap anirudhprasad-sap left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@Pavan-SAP Pavan-SAP merged commit 13e5f09 into main May 22, 2026
8 checks passed
@Pavan-SAP Pavan-SAP deleted the gofmt-fix branch May 22, 2026 07:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants