Skip to content

Conversation

@fmeum
Copy link
Member

@fmeum fmeum commented Sep 17, 2024

What type of PR is this?

Feature

What does this PR do? Why is it needed?

The default Go SDK with Bzlmod is kept at 1.22.7 as 1.23 is breaking in a number of ways and we don't want to force this onto every module as the default just yet.

Requires an update to golang.org/x/tools to avoid nogo silently failing to generate facts files with the new SDK.

Which issues(s) does this PR fix?

Other notes for review

The default Go SDK with Bzlmod is kept at 1.22.7.

Requires an update to golang.org/x/tools to avoid nogo silently failing to generate facts files with the new SDK.

Also requires an update to golang.org/x/net to fix bettercap/bettercap#1106.
@fmeum fmeum marked this pull request as ready for review September 18, 2024 14:48
@fmeum fmeum merged commit 8b8294b into master Sep 18, 2024
5 checks passed
@fmeum fmeum deleted the go1.23 branch September 18, 2024 21:34
peng3141 pushed a commit to peng3141/rules_go that referenced this pull request Sep 24, 2024
**What type of PR is this?**

Feature

**What does this PR do? Why is it needed?**

The default Go SDK with Bzlmod is kept at 1.22.7 as 1.23 is breaking in
a number of ways and we don't want to force this onto every module as
the default just yet.

Requires an update to golang.org/x/tools to avoid nogo silently failing
to generate facts files with the new SDK.

**Which issues(s) does this PR fix?**

**Other notes for review**
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