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.

Sign up for GitHub

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

Jump to bottom

cmd/go: mention go get in go help mod #44726

Closed
bukowa opened this issue Mar 2, 2021 · 5 comments
Closed

cmd/go: mention go get in go help mod #44726

bukowa opened this issue Mar 2, 2021 · 5 comments
Labels
Documentation GoCommand cmd/go help wanted NeedsFix The path to resolution is known, but the work has not been done.
Milestone
Backlog

Comments

@bukowa
Copy link

bukowa commented Mar 2, 2021

What version of Go are you using (go version)?

$ go version
go version go1.16

Does this issue reproduce with the latest release?

Yes

What did you do?

I were working with a module using command go mod; i expected that i can update dependency with commands:
go mod update -u <dependency>
go mod update <dependency>@commit

What did you expect to see?

I expected for this to work.
I expected for this to show in go help mod:

Usage:

        go mod <command> [arguments]

The commands are:

        download    download modules to local cache
        edit        edit go.mod from tools or scripts
        graph       print module requirement graph
        init        initialize new module in current directory
        tidy        add missing and remove unused modules
        vendor      make vendored copy of dependencies
        verify      verify dependencies have expected content
        why         explain why packages or modules are needed

What did you see instead?

After asking a question regarding this topic on golang slack channel i received help from Tim Heckman that stated:

yeah, the choice was made to keep it as go get so all the existing Gophers wouldn't need to relearn the command(s).

yeah, that was kept for historical purposes... that was how dependencies have always been gotten.

He nicely explained to me that i have to use:

go get <dependency>@commit
go get -u <dependency>

Which for me is confusing - i expected that subcommand to be related to go mod but - it's not in the help of that command - while it's clearly related to modules.
@bukowa bukowa changed the title subcommands related to "go mod" are not categorized under its "go help mod" subcommands related to "go mod" are not categorized under its "go help mod" ("go get") Mar 2, 2021
@go101
Copy link

go101 commented Mar 2, 2021 •
edited

Do you mean to use go mod update to depreciate go get? Not a bad idea IMO.

@bukowa
Copy link
Author

bukowa commented Mar 2, 2021

I don't know about depreciating go get but because that command is related to modules - i believe it should be included in go mod

@seankhliao seankhliao changed the title subcommands related to "go mod" are not categorized under its "go help mod" ("go get") cmd/go: mention go get in go help mod Mar 2, 2021
@seankhliao seankhliao added Documentation GoCommand cmd/go NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. labels Mar 2, 2021
@bcmills bcmills added NeedsFix The path to resolution is known, but the work has not been done. help wanted labels Mar 2, 2021
@bcmills bcmills added this to the Backlog milestone Mar 2, 2021
@gopherbot gopherbot removed the NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. label Mar 2, 2021
codykaup added a commit to codykaup/go that referenced this issue Dec 28, 2021
@codykaup
cmd/go: add go mod update command
9df3062 
This change adds a 'update' command to 'go mod' for updating module
dependencies.

Fixes golang#44726
codykaup added a commit to codykaup/go that referenced this issue Dec 28, 2021
@codykaup
cmd/go: add go mod update command
1140605 
This change adds an 'update' command to 'go mod' for updating module
dependencies.

Fixes golang#44726
@codykaup codykaup mentioned this issue Dec 28, 2021
Closed
@gopherbot
Copy link

Change https://golang.org/cl/374614 mentions this issue: cmd/go: add go mod update command

@mvdan
Copy link
Member

mvdan commented Dec 29, 2021

Are we actually suggesting that go mod should gain a command to update modules? The last comments mention that, and @bcmills added the NeedsFix label. But the Documentation label is also present, which tells me we haven't decided to add a command yet.

In terms of docs, I do think go get is already mentioned pretty well:

$ go help mod
Go mod provides access to operations on modules.

Note that support for modules is built into all the go commands,
not just 'go mod'. For example, day-to-day adding, removing, upgrading,
and downgrading of dependencies should be done using 'go get'.
See 'go help modules' for an overview of module functionality.

go help modules doesn't directly talk about go get, but it points to a few places that do mention it pretty often.

If you think the docs aren't clear enough, how do you suggest we improve them?

It's understandable that you expected go mod update to exist, but if you run go help mod just like the error from go mod update tells you, you'll be pointed at go get straight away.

Also, in terms of "just update all dependencies", see #28424.

@bukowa
Copy link
Author

bukowa commented Dec 14, 2023

It's understandable that you expected go mod update to exist, but if you run go help mod just like the error from go mod update tells you, you'll be pointed at go get straight away.

Yes I expected this tree to be clear, as where I learn very fast by looking at well structured data in key:value pair - not by having to execute 2 additional commands which my brain fogs a little specially as I am not native english speaker.

When working with structure data I would expect go mod update to work. One can think that there is no update command at all.

Usage:

        go mod <command> [arguments]

The commands are:

        download    download modules to local cache
        edit        edit go.mod from tools or scripts
        graph       print module requirement graph
        init        initialize new module in current directory
        tidy        add missing and remove unused modules
        vendor      make vendored copy of dependencies
        verify      verify dependencies have expected content
        why         explain why packages or modules are needed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Documentation GoCommand cmd/go help wanted NeedsFix The path to resolution is known, but the work has not been done.
Projects
None yet
Milestone
Backlog
Development

Successfully merging a pull request may close this issue.

cmd/go: add go mod update command codykaup/go
6 participants
@mvdan @bcmills @gopherbot @seankhliao @go101 @bukowa