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

Jump to bottom

all: TryBots don't complain about not gofmt code in the package #41352

Closed
alexbrainman opened this issue Sep 12, 2020 · 2 comments
Closed

all: TryBots don't complain about not gofmt code in the package #41352

alexbrainman opened this issue Sep 12, 2020 · 2 comments
Labels
FrozenDueToAge

Comments

@alexbrainman
Copy link
Member

I was doing codereview of

https://go-review.googlesource.com/c/go/+/250077

and discovered that patch set 2 was not gofmted, but TryBot did not complain about it.

I have asked CL author to gofmt his code. Is that something should be enforced by TryBots automatically? Maybe have a test that verifies that all appropriate code is gofmted.

Maybe it is OK to submit un formatted code. Feel free to close this issue then.

Thank you.

Alex
@tklauser
Copy link
Member

Also see #18548 which proposes something similar with additional checks (e.g. copyright headers).

@alexbrainman
Copy link
Member Author

Thanks @tklauser for the reference. It does indeed looks like a duplicate. So closing my issue.

Mind you, given #18548 is nearly 3 years old, does not give me much confidence. :-)

Alex

@golang golang locked and limited conversation to collaborators Sep 15, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
FrozenDueToAge
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@tklauser @gopherbot @alexbrainman