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

Dependancy updates to Go.mod packages #8822

Closed
wants to merge 1 commit into from

Conversation

bhaskarvilles
Copy link

I have checked the latest versions from package repositories and changed to best upgrades availabe.

Signed-off-by: Bhaskar hey@bhaskar.email

Signed-off-by: Bhaskar <hey@bhaskar.email>
@Jorropo
Copy link
Contributor

Jorropo commented Mar 24, 2022

@bhaskarvilles the tests are failing you need to go mod tidy too after updating.

@bhaskarvilles
Copy link
Author

@bhaskarvilles the tests are failing you need to go mod tidy too after updating.

I got no errors in local branch, but the upgrades was clear and no conflicts with base branch, why it happens with CI ?

@Jorropo
Copy link
Contributor

Jorropo commented Mar 25, 2022

Because your local tests dont check for go mod tidy.

@BigLep BigLep marked this pull request as draft March 25, 2022 15:27
@BigLep
Copy link
Contributor

BigLep commented Mar 25, 2022

@bhaskarvilles: thanks for the PR. We usually do this as part of our release process. We will get that formulated in #8576 . I'm closing this as a result.

@BigLep BigLep closed this Mar 25, 2022
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.

3 participants