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 rules_go & Bazel deps #1973

Merged
merged 5 commits into from
Nov 8, 2024
Merged

Bump rules_go & Bazel deps #1973

merged 5 commits into from
Nov 8, 2024

Conversation

fmeum
Copy link
Member

@fmeum fmeum commented Nov 6, 2024

What type of PR is this?

Deps

What package or component does this PR mostly affect?

all

What does this PR do? Why is it needed?

Bumping the compatibility SDK is needed due to changes to golang.org/x/tools that require Go 1.19 (types.Func.Origin()) and, temporarily, due to non-hermeticity in rules_go's GoToolchainBinaryBuild.

Which issues(s) does this PR fix?

Fixes #

Other notes for review

Not updating to rules_go 0.50.1 quite yet as that runs into issues with nogo.

Bumping the compatibility SDK is needed due to changes to `golang.org/x/tools` that require Go 1.19 (`types.Func.Origin()`).
@fmeum fmeum marked this pull request as draft November 6, 2024 16:12
@fmeum fmeum force-pushed the update-deps branch 2 times, most recently from 49b2c4d to a82e8b9 Compare November 6, 2024 17:04
@fmeum fmeum force-pushed the update-deps branch 4 times, most recently from 3b1fa00 to 21cdc73 Compare November 7, 2024 20:08
@fmeum fmeum marked this pull request as ready for review November 8, 2024 09:25
@fmeum fmeum merged commit bf8c993 into master Nov 8, 2024
16 checks passed
@fmeum fmeum deleted the update-deps branch November 8, 2024 18:10
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.

1 participant