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

proposal: gddo: remove `Uses` links to sourcegraph #22203

Closed
adams-sarah opened this Issue Oct 10, 2017 · 3 comments

Comments

Projects
None yet
3 participants
@adams-sarah
Contributor

adams-sarah commented Oct 10, 2017

When the Uses links first appeared on godoc.org, I believe they used to show uses of the identifier in open-source code external to its own package via SourceGraph.
This used to be really helpful - an easy way to find examples.

Maybe SourceGraph's pricing model changed, such that you need to be a paying customer to access uses in external code.

In any case, the links currently show uses in the current package, which is not very helpful (IMHO). When uses of exported identifiers in the current package do come up, they are typically just being wrapped lightly.

I propose we remove these links. Reduce clutter of codebase + UI.

cc @shurcooL @shantuo

@gopherbot gopherbot added this to the Proposal milestone Oct 10, 2017

@gopherbot gopherbot added the Proposal label Oct 10, 2017

@dmitshur

This comment has been minimized.

Show comment
Hide comment
@dmitshur

dmitshur Oct 11, 2017

Member

The gddo project has its own issue tracker at https://github.com/golang/gddo/issues, mind moving this proposal there?

Member

dmitshur commented Oct 11, 2017

The gddo project has its own issue tracker at https://github.com/golang/gddo/issues, mind moving this proposal there?

@adams-sarah

This comment has been minimized.

Show comment
Hide comment
@adams-sarah

adams-sarah Oct 11, 2017

Contributor

did not know that, thx.
there is another one in the issue tracker i just copied the prefix for. i'll ping them too.

Contributor

adams-sarah commented Oct 11, 2017

did not know that, thx.
there is another one in the issue tracker i just copied the prefix for. i'll ping them too.

@adams-sarah

This comment has been minimized.

Show comment
Hide comment
@adams-sarah
Contributor

adams-sarah commented Oct 11, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment