-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Documentation on contributing #1647
Conversation
Hello, could you explain a bit more the change? |
sorry, didn't see that tit was changed from covet to unparam
@butuzov Yeah, could you please explain why did you change the example from |
Hi Guys. I had run into an issue yesterday, that This is minor change to the website docs and can be left in place, but I guess we will skip the initial frustration of the person reading at the website "For example, What I have missed is that there is another option for
|
Again, this quite a minor thing, and not critical. |
`govet` is replaced by `nakedret` as example of the function that has only one option in settings.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This PR fix the reference to
govet
in contributing docs as linter that has one option. Not critical, just cleaning.