Skip to content
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

cmd/go: remove go mod fix command #26831

Closed
rajender opened this issue Aug 6, 2018 · 2 comments
Closed

cmd/go: remove go mod fix command #26831

rajender opened this issue Aug 6, 2018 · 2 comments
Labels
FrozenDueToAge modules NeedsFix The path to resolution is known, but the work has not been done. release-blocker
Milestone

Comments

@rajender
Copy link
Contributor

rajender commented Aug 6, 2018

As discussed in https://go-review.googlesource.com/c/go/+/126655 we should remove 'go mod fix' command as it's functionality is part of many commands and overlaps with 'go mod tidy'.

As per Russ in above CL,

"Honestly the most useful thing about fix is its doc comment,
since otherwise that part of all the other commands is not documented anywhere.
I don't expect people to actually use the operation.
If we dropped 'go mod fix' people could use 'go list -m' or almost anything else.

I need to write a 'go help go.mod'. If I do that, then maybe we can move the
fix discussion into that and drop 'go mod fix' entirely. Will leave for now though."

@rajender
Copy link
Contributor Author

rajender commented Aug 6, 2018

@gopherbot, please add labels modules

@bcmills bcmills added the NeedsFix The path to resolution is known, but the work has not been done. label Aug 7, 2018
@bcmills bcmills added this to the Go1.11 milestone Aug 7, 2018
@gopherbot
Copy link

Change https://golang.org/cl/129682 mentions this issue: cmd/go: remove go mod fix, add go help go.mod

@golang golang locked and limited conversation to collaborators Aug 18, 2019
@rsc rsc removed their assignment Jun 23, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
FrozenDueToAge modules NeedsFix The path to resolution is known, but the work has not been done. release-blocker
Projects
None yet
Development

No branches or pull requests

4 participants