-
Notifications
You must be signed in to change notification settings - Fork 279
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
package-comments flags main as of 1.2.2 #740
Comments
Hi @twmb, thanks for filling the issue. Well, even if documenting
and it also provides an example for a /*
Gofmt formats Go programs.
It uses tabs for indentation and blanks for alignment.
Alignment assumes that an editor is using a fixed-width font.
[...]
When gofmt reads from standard input, [...]
formatted by piping them through gofmt.
*/
package main As a reference, standard GO tools have documented main packages. Must |
We have meet this problem, too. but we are not main package.
|
I'm open to it, but I think it's worth a quick survey of the packages that already use revive to see if they currently document main. I never have -- I document my main packages elsewhere. As well, main packages do not follow the common |
Hi @hawkingrei , I've checked the code you point in your comment and I've seen there is no documentation for the |
|
This might not be a bug, but I'd like to clarify if this is intentional. It's not common to document main packages, I personally plan to disable the rule if it continues to lint documenting main.
The text was updated successfully, but these errors were encountered: