Can this be as well added in .golangci.yml?

Didn't we have issues with this on Osmosis? What does this buy us over goimports which is natively supported by gopls and vscode?

@alexanderbez yep, we did.

I figure if we do it once in a while, we can get the benefits without the pain of having it in ci.

In ci, it could cause pain because then everyone must

gofumpt -w -l . every time

@ValarDragon mentioned it could be rough for first timers and I agree.

Also, I think it makes the code look and feel nicer.

Maybe do once in a while for a bit, and when the tooling/practice is more mature we add it to ci.

For example if it ran automatically when users:

golangci-lint run --fix ./... that'd be awesome

Perhaps we can add gofumpt'ing the entire code to a script that we execute pre-releases and prior to making new major release branches?

I also think this is the best idea for now (suggestion from the osmosis issue)

@faddat do you want to add this in the release process?

@marbar3778 Sure!

@marbar3778 I've added gofumpt to the release process description document.

@julienrbrt -- I cannot exactly explain / justify changes maade by gofumpt -w -l .

Overall, I like what it does for the appearance and standardization of code, and you can check it out in more detail here:

https://github.com/mvdan/gofumpt

Sorry bout that

@julienrbrt -- I cannot exactly explain / justify changes maade by gofumpt -w -l .

Overall, I like what it does for the appearance and standardization of code, and you can check it out in more detail here:

https://github.com/mvdan/gofumpt

Sorry bout that

I understand, I use the tool as part of my workflow as well.
What I meant is that you can tweak how gofumpt will format lines by improving the formatting yourself first. My comments were suggestions where to do that:

For instance, in x/distribution/types/query.go we have:

func NewQueryDelegatorTotalRewardsResponse(rewards []DelegationDelegatorReward,
	total sdk.DecCoins) QueryDelegatorTotalRewardsResponse {
	return QueryDelegatorTotalRewardsResponse{Rewards: rewards, Total: total}
}

running gofumpt -w -l . on that will give us

func NewQueryDelegatorTotalRewardsResponse(rewards []DelegationDelegatorReward,
	total sdk.DecCoins,
) QueryDelegatorTotalRewardsResponse {
	return QueryDelegatorTotalRewardsResponse{Rewards: rewards, Total: total}
}

which is weird.

If before running gofumpt, we improve the styling to

func NewQueryDelegatorTotalRewardsResponse(rewards []DelegationDelegatorReward, total sdk.DecCoins) QueryDelegatorTotalRewardsResponse {
	return QueryDelegatorTotalRewardsResponse{Rewards: rewards, Total: total}
}

nothing will change, and we get, imo a better styling than with gofumpt.

I think it worth it to do that manual improving where we see gofumpt format the code with a one argument line.

I think I agree @julienrbrt. I should either be one line or each arg in a separate line. What gofumpt produces,

func NewQueryDelegatorTotalRewardsResponse(rewards []DelegationDelegatorReward,
	total sdk.DecCoins,
) QueryDelegatorTotalRewardsResponse {
	return QueryDelegatorTotalRewardsResponse{Rewards: rewards, Total: total}
}

Is not great. I'd rather see:

func NewQueryDelegatorTotalRewardsResponse(
	rewards []DelegationDelegatorReward,
	total sdk.DecCoins,
) QueryDelegatorTotalRewardsResponse {
	return QueryDelegatorTotalRewardsResponse{Rewards: rewards, Total: total}
}

@faddat Do you mind If I fix these nits myself in your branch?

I don't mind at all :).

I am also wondering how else we can improve. Your nits were real.

aha!

OK, now that I have re-read your comments, it makes sense.

1st update:
I think I can find time tomorrow, or can give you access to our repo. Thanks a bunch for the review and suggestions!

2nd update:
thanks for the pr!

Thanks for updating!