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

gofumpt -w -l #1468

Closed
wants to merge 1 commit into from
Closed

gofumpt -w -l #1468

wants to merge 1 commit into from

Conversation

faddat
Copy link
Contributor

@faddat faddat commented May 3, 2022

Gorgeous code formatting

go install mvdan.cc/gofumpt@latest
gofumpt -w -l .

@codecov
Copy link

codecov bot commented May 3, 2022

Codecov Report

Merging #1468 (538d5dc) into main (c934600) will increase coverage by 0.08%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##             main    #1468      +/-   ##
==========================================
+ Coverage   15.68%   15.76%   +0.08%     
==========================================
  Files          10       10              
  Lines        1371     1364       -7     
==========================================
  Hits          215      215              
+ Misses       1145     1138       -7     
  Partials       11       11              

@okwme
Copy link
Contributor

okwme commented May 20, 2022

thanks for the suggestion @faddat but we're going to stay consistent with SDK for now

@okwme okwme closed this May 20, 2022
@faddat
Copy link
Contributor Author

faddat commented May 27, 2022

Billy the SDK uses gofumpt

cosmos/cosmos-sdk#11839

Let's stay consistent with the SDK

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