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

Avoid loading resources from the network on make format #192

Merged
merged 6 commits into from
Oct 29, 2019

Conversation

jkohen
Copy link
Contributor

@jkohen jkohen commented Oct 28, 2019

This removes network loads until go fmt stops trying to download those resources. golang/go#27841 has more information.

This removes network loads until `go fmt` stops trying to download those resources. golang/go#27841 has more information.
@jkohen jkohen self-assigned this Oct 28, 2019
Added dependency between coverage and format, just to test execution from Travis.
@jkohen jkohen changed the title Invoke go fmt with mod=vendor Prevent network requests on go fmt Oct 28, 2019
@jkohen jkohen changed the title Prevent network requests on go fmt Avoid loading resources from the network on make format Oct 28, 2019
@jkohen jkohen requested a review from a team October 28, 2019 21:54
@ghost ghost requested review from igorpeshansky and removed request for a team October 28, 2019 21:54
Makefile Show resolved Hide resolved
@jkohen jkohen requested a review from igorpeshansky October 29, 2019 13:35
Copy link
Member

@igorpeshansky igorpeshansky left a comment

Choose a reason for hiding this comment

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

LGTM :shipit:

Makefile Show resolved Hide resolved
@jkohen jkohen merged commit 1266a83 into master Oct 29, 2019
@jkohen jkohen deleted the go-fmt-mod-vendor branch October 29, 2019 16:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants