-
Notifications
You must be signed in to change notification settings - Fork 18.3k
Closed
Labels
FrozenDueToAgeProposalProposal-AcceptedToolsThis label describes issues relating to any tools in the x/tools repository.This label describes issues relating to any tools in the x/tools repository.
Milestone
Description
The documentation for x/tools/cmd/cover
says:
Deprecated: For Go releases 1.5 and later, this tool lives in the standard repository. The code here is not maintained.
I count 11 CLs updating the package since that notice was first added in CL 9539.
Keeping the code around clearly isn't doing anyone any good — at the very least, it adds noise in git grep
results (which is how I came to notice it in the first place).
I propose that we give it the same treatment as x/exp/winfsnotify
: carve it out into a nested module, tag that module cmd/cover/v0.1.0-deprecated
, and then delete it from the x/tools
repo at HEAD.
mvdan, earthboundkid, komuw and dmitshur
Metadata
Metadata
Assignees
Labels
FrozenDueToAgeProposalProposal-AcceptedToolsThis label describes issues relating to any tools in the x/tools repository.This label describes issues relating to any tools in the x/tools repository.