Skip to content

style: gofumpt all the things#9829

Merged
calmh merged 1 commit intosyncthing:mainfrom
calmh:jb/gofumpt
Nov 19, 2024
Merged

style: gofumpt all the things#9829
calmh merged 1 commit intosyncthing:mainfrom
calmh:jb/gofumpt

Conversation

@calmh
Copy link
Copy Markdown
Member

@calmh calmh commented Nov 19, 2024

Literally gofumpt -w . from the top level dir. Guaranteed to be minor style changes only and nothing else.

@imsodin per request?

Literally `gofumpt -w .` from the top level dir. Guaranteed to be minor
style changes only and nothing else.
Copy link
Copy Markdown
Member

@acolomb acolomb left a comment

Choose a reason for hiding this comment

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

Yes, please.

@calmh
Copy link
Copy Markdown
Member Author

calmh commented Nov 19, 2024

Okidokie then

@calmh
Copy link
Copy Markdown
Member Author

calmh commented Nov 19, 2024

Overriding policy-bot because style: is not one of the approved prefixes, but maybe it should be (or maybe it shouldn't and we shouldn't do more of these...)

@calmh calmh merged commit e82ed6e into syncthing:main Nov 19, 2024
calmh added a commit to calmh/syncthing that referenced this pull request Nov 19, 2024
* main:
  style: gofumpt all the things (syncthing#9829)
@acolomb
Copy link
Copy Markdown
Member

acolomb commented Nov 19, 2024

Overriding policy-bot because style: is not one of the approved prefixes

The PR content is about code style, but the task itself falls under the "chore" category alright.

@calmh calmh added this to the v1.28.1 milestone Nov 25, 2024
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