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

Comments

@ajstarks
Copy link
Contributor

@ajstarks ajstarks commented Jan 18, 2021

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 the pkgsite label Jan 18, 2021
@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
Contributor

@seankhliao seankhliao commented Jan 18, 2021

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

@ajstarks ajstarks commented Jan 19, 2021

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
Contributor

@julieqiu julieqiu commented Jan 19, 2021

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.

@ajstarks
Copy link
Contributor Author

@ajstarks ajstarks commented Jan 19, 2021

@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 jba commented Jan 25, 2021

Removed.

@jba jba closed this Jan 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
5 participants