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

Run "go mod tidy" #6920

Closed
wants to merge 1 commit into from
Closed

Run "go mod tidy" #6920

wants to merge 1 commit into from

Conversation

dweitzman
Copy link
Member

On motivation for this is that "github.com/google/shlex" is missing from the list of deps.

Seems like we should be building with -mod=readonly, which would have caught this by refusing to allow unversioned dependencies

Signed-off-by: David Weitzman dweitzman@pinterest.com

Signed-off-by: David Weitzman <dweitzman@pinterest.com>
@deepthi
Copy link
Member

deepthi commented Oct 20, 2020

I ran into this and included it in #6904. I'm holding off on merging anything right now until docker builds are fixed (and we can get CI green again).

@deepthi deepthi requested a review from dkhenry October 22, 2020 01:50
@deepthi
Copy link
Member

deepthi commented Oct 22, 2020

This change seems to be more extensive than what I had in #6904. Requesting a review from @dkhenry who is familiar with the Azure plugin.

@dweitzman
Copy link
Member Author

This is out of date and some version of it is probably already merged, so closing it

@dweitzman dweitzman closed this Nov 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants