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

internal: update golint to use new url #2393

Merged
merged 1 commit into from
Oct 31, 2018

Conversation

jeanbza
Copy link
Member

@jeanbza jeanbza commented Oct 22, 2018

(the repo moved)

(I have zero experience with go modules and do not have a good idea of whether more changes are needed to go.mod)

@jeanbza
Copy link
Member Author

jeanbza commented Oct 29, 2018

cc @jadekler

@jeanbza
Copy link
Member Author

jeanbza commented Oct 31, 2018

PTAL (used go mod tidy)

Copy link
Member

@dfawley dfawley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! 👍

@jeanbza jeanbza merged commit 6572a80 into grpc:master Oct 31, 2018
gopherbot pushed a commit to golang/build that referenced this pull request Mar 14, 2019
Version v1.16.0 of the grpc module is known to be bad (because it
doesn't include the fix from grpc/grpc-go#2393), so we don't want
to require it, even indirectly.

Commands run:

	go mod edit -droprequire=google.golang.org/grpc
	go mod tidy
	go test ./...

Updates golang/go#30833

Change-Id: I549dd5276ae19aa89c18cf2e9c981e81b8ffdd11
Reviewed-on: https://go-review.googlesource.com/c/build/+/167597
Run-TryBot: Dmitri Shuralyov <dmitshur@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
@lock lock bot locked as resolved and limited conversation to collaborators Apr 30, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants