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 x/oauth2 & x/crypto (#33704) #33727

Merged

Conversation

techknowlogick
Copy link
Member

Backport dep bump

@GiteaBot GiteaBot added this to the 1.23.5 milestone Feb 26, 2025
@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Feb 26, 2025
@pull-request-size pull-request-size bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Feb 26, 2025
@techknowlogick techknowlogick changed the title bump x/oauth2 & x/crypto bump x/oauth2 & x/crypto (#33704) Feb 26, 2025
@GiteaBot GiteaBot added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Feb 26, 2025
go 1.23
go 1.23.0

toolchain go1.23.6
Copy link
Member

@silverwind silverwind Feb 26, 2025

Choose a reason for hiding this comment

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

I don't know why go wants to introduce this toolchain directive and the .0 version here, but it seems like a go bug to me. See golang/go#65847 (comment) for my findings.

Copy link
Member

Choose a reason for hiding this comment

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

Likely the change itself is bening, toolchain is only a "suggestion" to the go compiler.

@silverwind
Copy link
Member

The build failure seems persistent and odd. It looks as if google/go-licenses does not create the output directory .go-licenses, but when I try the same locally, it works.

@silverwind
Copy link
Member

Pushed an attempt to fix license build with GOOS=linux.

@silverwind
Copy link
Member

#33747 will fix it, merge this after that one.

Copy link
Member

@silverwind silverwind left a comment

Choose a reason for hiding this comment

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

Should be mergeable after #33747 has backported.

@GiteaBot GiteaBot removed the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Feb 28, 2025
@GiteaBot GiteaBot added the lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. label Feb 28, 2025
@techknowlogick techknowlogick merged commit d634e75 into go-gitea:release/v1.23 Feb 28, 2025
25 of 26 checks passed
@techknowlogick techknowlogick deleted the go-dep-bump-1.23 branch February 28, 2025 17:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. modifies/dependencies size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants