You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on May 9, 2021. It is now read-only.
If some of the methods of the type must have pointer receivers, the rest should too, so the method set is consistent regardless of how the type is used
Would it be reasonable to have a rule for this? It is particularly important for types that contain fields for which copying breaks a contract, like atomic.Value. If I've defined functions of Foo to be pointer receivers specifically because Foo contains an atomic type, I'd want a warning if someone acidentally adds a function with a value receiver.
The text was updated successfully, but these errors were encountered:
Thank you for submitting this issue! As per golang/go#38968, we are freezing and deprecating golint. There's no drop-in replacement to golint per se, but you should find that Staticcheck works well in encouraging good Go code, much like golint did in the past, since it also includes style checks. There's always gofmt and go vet too, of course.
If you would like to contribute further, I'd encourage you to engage Staticcheck's issue tracker or look at vet's open issues, as they are both actively maintained. If you have an idea that doesn't fit into either of those tools, you could look at other Go linters, or write your own - these days it's fairly straightforward with go/analysis.
To help avoid confusion, I'm closing all issues before we freeze the repository. If you have any feedback, you can leave a comment on the proposal thread where it was decided to deprecate golint - though note that the proposal has been accepted for nearly a year. Thanks!
https://golang.org/doc/faq#methods_on_values_or_pointers says:
Would it be reasonable to have a rule for this? It is particularly important for types that contain fields for which copying breaks a contract, like
atomic.Value
. If I've defined functions ofFoo
to be pointer receivers specifically becauseFoo
contains an atomic type, I'd want a warning if someone acidentally adds a function with a value receiver.The text was updated successfully, but these errors were encountered: