-
-
Notifications
You must be signed in to change notification settings - Fork 110
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 -s no longer works #174
Milestone
Comments
Ah, I see that it's described here Well, OK. |
That's fair - a bit unnecessary to remove the flags outright. We can remove the code and UX complexity but still leave the flags declared as a warning/error. For |
Oiyoo
added a commit
to Oiyoo/gofumpt
that referenced
this issue
Dec 6, 2021
Oiyoo
added a commit
to Oiyoo/gofumpt
that referenced
this issue
Dec 7, 2021
mvdan
pushed a commit
that referenced
this issue
Dec 12, 2021
This way, users calling -s or -r get a helpful message. The use of -s can still let the tool work, for now. Fixes #174.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
It seems
gofumpt -s
no longer works, which broke some of my scripts.The readme claims that gofumpt is
while this broke the backwards compatibility.
I understand that
-s
is now default, but the flag should IMO still exist (and do nothing?).The text was updated successfully, but these errors were encountered: