Skip to content
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

style: reformat all go files #437

Merged
merged 10 commits into from
Jun 19, 2024
Merged

style: reformat all go files #437

merged 10 commits into from
Jun 19, 2024

Conversation

developStorm
Copy link
Member

And activate go formatter workflow

@developStorm developStorm added the go Pull requests that update Go code label May 21, 2024
@developStorm developStorm requested a review from a team May 21, 2024 22:47
@developStorm developStorm self-assigned this May 21, 2024
@developStorm developStorm marked this pull request as ready for review May 21, 2024 22:47
@developStorm developStorm marked this pull request as draft May 21, 2024 22:51
@developStorm developStorm force-pushed the build/go-formatter branch 3 times, most recently from d1cc022 to 70122bd Compare May 21, 2024 23:03
@developStorm developStorm marked this pull request as ready for review May 21, 2024 23:04
@developStorm developStorm force-pushed the build/go-formatter branch 8 times, most recently from 2c7d103 to cee0b27 Compare May 22, 2024 04:47
Copy link
Contributor

@phillip-stephens phillip-stephens left a comment

Choose a reason for hiding this comment

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

Overall looks good, thanks for taking this up @developStorm !

One thing, could we add a make lint similar to here so it's easier to do this locally without hunting for the gofmt command?

@developStorm
Copy link
Member Author

Thanks for reviewing @phillip-stephens! Implemented in 18d5069. I did only go fmt since linter is out of scope of this PR...

Copy link
Contributor

@phillip-stephens phillip-stephens left a comment

Choose a reason for hiding this comment

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

Ran gofmt myself and confirmed the change-set is the same. LGTM

@phillip-stephens phillip-stephens merged commit 6cdc779 into master Jun 19, 2024
4 checks passed
@phillip-stephens phillip-stephens deleted the build/go-formatter branch June 19, 2024 02:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
go Pull requests that update Go code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants