cobra.dev update required

spf13 / cobra

A Commander for modern Go CLI interactions

https://cobra.dev

Apache License 2.0

36.93k stars 2.81k forks source link

cobra.dev update required #1674

Open theishanbh opened 2 years ago

theishanbh commented 2 years ago

➜ borzoi go get -u github.com/spf13/cobra/cobra go: module github.com/spf13/cobra@upgrade found (v1.4.0), but does not contain package github.com/spf13/cobra/cobra

While trying to install cobra, i ran into the above error. The documentation should be fixed in the website so that it can be fixed correctly. I realised I was able to do it running the below command :-

borzoi go get -u github.com/spf13/cobra/cobra@latest

Hence, the website must be updated.

jpmcb commented 2 years ago

Where are you seeing this? Our installing documentation has the correct command in this section: https://github.com/spf13/cobra#installing

go get -u github.com/spf13/cobra@latest

Also note that the cobra/cobra module no longer exists in this repository. It has moved to https://github.com/spf13/cobra-cli

theishanbh commented 2 years ago

I wanted to reference, https://cobra.dev/.

umarcor commented 2 years ago

FTR, #1245 is waiting for #1428 to be merged.

github-actions[bot] commented 2 years ago

The Cobra project currently lacks enough contributors to adequately respond to all issues. This bot triages issues and PRs according to the following rules:

After 60d of inactivity, lifecycle/stale is applied. - After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied and the issue is closed. You can:
Make a comment to remove the stale label and show your support. The 60 days reset. - If an issue has lifecycle/rotten and is closed, comment and ask maintainers if they'd be interseted in reopening

PaluMacil commented 1 year ago

this is probably worth reopening

umarcor commented 1 year ago

FTR, I just rebased #1428 to remove merge conflicts.

PaluMacil commented 1 year ago

I reviewed it, though since I don't have write privileges, I'm not sure it matters. Thanks @umarcor

tlarnold10 commented 3 weeks ago

@theishanbh looks like a PR was merged for this. Can we close this issue?