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

x/all: READMEs are out of date #42985

Open
FiloSottile opened this issue Dec 3, 2020 · 0 comments
Open

x/all: READMEs are out of date #42985

FiloSottile opened this issue Dec 3, 2020 · 0 comments
Labels
NeedsDecision Feedback is required from experts, contributors, and/or the community before a change can be made.
Milestone
Unreleased

Comments

@FiloSottile
Copy link
Contributor

The current README of the x-repos dates back to pre-modules and pre-PRs times.

https://github.com/golang/crypto/blob/master/README.md

We should update them to

  1. replace the install instructions with just go get golang.org/x/crypto with no -u (because we don't want people to pull up all transitive deps if they just want x/crypto), no /... because we don't want to install any commands, and no $GOPATH because that's deprecated
  2. mention that we accept GitHub PRs as well as Gerrit CLs

/cc @golang/release @julieqiu
@FiloSottile FiloSottile added the NeedsDecision Feedback is required from experts, contributors, and/or the community before a change can be made. label Dec 3, 2020
@gopherbot gopherbot added this to the Unreleased milestone Dec 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
NeedsDecision Feedback is required from experts, contributors, and/or the community before a change can be made.
Projects
None yet
Milestone
Unreleased
Development

No branches or pull requests
2 participants
@FiloSottile @gopherbot