pre-commit-golang icon indicating copy to clipboard operation
pre-commit-golang copied to clipboard
verified publisher icon dnephin

Change go-fmt and goimports output

Open sergio-u opened this issue 5 years ago • 3 comments

Changing how these hooks report their status for improving cross-platform support. In particular, this should fix #53

Additionally, I've set go-mod-tidy to run on changes to .mod and .sum files besides *.go, but I'm unsure if this is actually required.

sergio-u avatar Oct 01 '20 19:10 sergio-u

Hi, I'm curious: What would it take for this PR to be merged? I'd be glad to help.

nilsalex avatar Jul 28 '21 13:07 nilsalex

Looking forward the PR to be merged, i am suffering gofmt hook's failure in windows .

whotakekajibu avatar Oct 20 '21 06:10 whotakekajibu

Hi, I'm curious: What would it take for this PR to be merged? I'd be glad to help.

Looks like @dnephin's comment needs to be addressed to merge this: https://github.com/dnephin/pre-commit-golang/pull/60#discussion_r511629276

echarrod avatar Jun 27 '22 09:06 echarrod

We merged some changes in #97 which I think had the same goal, so hopefully this problem is fixed now.

dnephin avatar Nov 26 '22 16:11 dnephin