cmd/vet: remove Peek from methods check #19719
Labels
FrozenDueToAge
NeedsDecision
Feedback is required from experts, contributors, and/or the community before a change can be made.
Milestone
vet's methods check ensures that methods with a canonical name have the right signature. This includes:
But it's not clear to me why the unexported
image.reader
interface is important enough to set the canonical interpretation of Peek.I see three Peeks in the standard library:
Of these, only
bufio.Reader
matches. AnotherPeek
that comes to mind is from Russ's article Off to the Races:I think we should remove
Peek
from the list of canonical method names.The CL is trivial; this issue is to ask for approval.
@robpike
The text was updated successfully, but these errors were encountered: