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

Bump versions: Golang dependencies #6

Merged
merged 1 commit into from
Feb 6, 2024
Merged

Bump versions: Golang dependencies #6

merged 1 commit into from
Feb 6, 2024

Conversation

bgandon
Copy link

@bgandon bgandon commented Feb 6, 2024

This is the PR telia-oss#275 that I've ported to this fork.

In this work that was don on 2022-02-02, I've bumped the Golang dependencies using Go v1.17. Worth to notice, the GitHub API is bumped from v28 to v42. This is important because currently this github-pr-resource cannot push any PR status anymore, producing a 404 error, due to deprecated API endpoint I guess.

Golang and Alpine versions are not touched here, because they are controlled by the Concourse pipeline.

You'll not that the go.sum file is way larger as before. This comes from bumping the required dependencies with go get invocations, and then running go mod tidy. I've tried to re-run these commands from scratch in case it would produce a smaller set of dependencies, and this has reached the exact same result with go.mod and go.sum.

@bgandon
Copy link
Author

bgandon commented Feb 6, 2024

Build and tests are passing ✅

@bgandon bgandon merged commit 0ba3ae9 into main Feb 6, 2024
@ctlong ctlong deleted the bump-versions branch February 6, 2024 19:09
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.

None yet

1 participant