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

x/pkgsite: incorrect item for github.com/ajstarks/decksh #43754

Closed
ajstarks opened this issue Jan 18, 2021 · 5 comments
Closed

x/pkgsite: incorrect item for github.com/ajstarks/decksh #43754

ajstarks opened this issue Jan 18, 2021 · 5 comments
Labels
FrozenDueToAge pkgsite WaitingForInfo Issue is not actionable because of missing required information, which needs to be provided.
Milestone

Comments

@ajstarks
Copy link
Contributor

What is the URL of the page with the issue?

https://pkg.go.dev/github.com/ajstarks/deck/cmd/dchart

What is your user agent?

Mozilla/5.0 (X11; Fedora; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/87.0.4280.141 Safari/537.36

Screenshot

image

What did you do?

The path is incorrect -- it points to a defunct command when it should be the library at github.com/ajstarks/decksh

What did you expect to see?

The correct module for github.com/ajstarks/decksh

What did you see instead?

the item is incorrectly under github.com/ajstarks/deck > cmd > dchart

@gopherbot gopherbot added this to the Unreleased milestone Jan 18, 2021
@seankhliao seankhliao changed the title x/pkgsite: incorrect item for github.com/ajstarks/decksh, please update x/pkgsite: incorrect item for github.com/ajstarks/decksh Jan 18, 2021
@seankhliao
Copy link
Member

It appears you have 2 repos https://github.com/ajstarks/deck and https://github.com/ajstarks/decksh

The warning label in the version identifier already indicates that it is not part of the latest version of deck, and it doesn't show up in the listing from the latest revision: https://pkg.go.dev/github.com/ajstarks/deck/cmd

@ajstarks
Copy link
Contributor Author

Note that https://pkg.go.dev/github.com/ajstarks/deck/cmd should be removed. The core issue is that decksh won't be built because it cannot find dchart.

@julieqiu
Copy link
Member

Note that https://pkg.go.dev/github.com/ajstarks/deck/cmd should be removed.

Would you like us to remove that package entirely from the site, per https://go.dev/about#removing-a-package?

If you are the author of a package and would like to have it removed from pkg.go.dev, please file an issue on the Go Issue Tracker with the path that you want to remove.

Note that we can only remove a module entirely from the site. We cannot remove it just for specific versions.

@julieqiu julieqiu added the WaitingForInfo Issue is not actionable because of missing required information, which needs to be provided. label Jan 19, 2021
@ajstarks
Copy link
Contributor Author

@julieqiu yes, please remove it as per your site. I suspect this old package is causing a clash that is preventing other module aware packages from building. (you cannot build decksh because it depends on dchart which it cannot find.)

@jba
Copy link
Contributor

jba commented Jan 25, 2021

Removed.

@jba jba closed this as completed Jan 25, 2021
@golang golang locked and limited conversation to collaborators Jan 25, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
FrozenDueToAge pkgsite WaitingForInfo Issue is not actionable because of missing required information, which needs to be provided.
Projects
None yet
Development

No branches or pull requests

5 participants