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.

Sign up for GitHub

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

Jump to bottom

chore: run gofumpt #1353

Merged
merged 2 commits into from
Dec 3, 2024
Merged

chore: run gofumpt #1353

merged 2 commits into from
Dec 3, 2024

Conversation

sashamelentyev
Copy link
Contributor

@sashamelentyev sashamelentyev commented Dec 1, 2024 •
edited
Loading

Run gofumpt -w . command.

@sashamelentyev sashamelentyev changed the title chore: gofumpt -w . chore: gofumpt -w Dec 1, 2024
tdakkota
tdakkota approved these changes Dec 2, 2024
View reviewed changes
Copy link
Member

@tdakkota tdakkota left a comment

Choose a reason for hiding this comment

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

LGTM, but commit linter is unhappy with dot in the commit title.

Let's change title to chore: run gofumpt and move actual command to commit body.

@sashamelentyev sashamelentyev changed the title chore: gofumpt -w chore: run gofumpt Dec 2, 2024
@sashamelentyev
Copy link
Contributor Author

@tdakkota done

@tdakkota tdakkota merged commit 35d970b into ogen-go:main Dec 3, 2024
14 checks passed
@sashamelentyev sashamelentyev deleted the gofumpt branch December 3, 2024 07:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tdakkota tdakkota tdakkota approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sashamelentyev @tdakkota