-
Notifications
You must be signed in to change notification settings - Fork 83
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
Add support for Go 1.21 #417
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. My team is using this patch in our Go monorepo for over a month now.
This works for old and new go versions and passes the unit tests on my machine.
Just a very gentle reminder that this PR is required for go 1.21 support and can be merged @benradf @aherrmann
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry for the delay in reviewing. Looks good 👍
@Mergifyio rebase |
Go 1.21 added a requirement for a 'go.env' file. Without that file, Go will throw an error like: ``` Error in fail: failed to fetch org_golang_google_grpc_cmd_protoc_gen_go_grpc: fetch_repo: google.golang.org/grpc/cmd/[email protected]: GOPROXY list is not the empty string, but contains no entries ``` This commit adds it to the SDK exposed by rules_nixpkgs so that it is compatible with rules_go. See: - bazel-contrib/rules_go#3665 - bazel-contrib/rules_go#3666
✅ Branch has been successfully rebased |
Using go 1.21 fails with `GOPROXY list is not the empty string, but contains no entries`. See tweag/rules_nixpkgs#417
Using go 1.21 fails with `GOPROXY list is not the empty string, but contains no entries`. See tweag/rules_nixpkgs#417
Using go 1.21 fails with `GOPROXY list is not the empty string, but contains no entries`. See tweag/rules_nixpkgs#417
Using go 1.21 fails with `GOPROXY list is not the empty string, but contains no entries`. See tweag/rules_nixpkgs#417
Using go 1.21 fails with `GOPROXY list is not the empty string, but contains no entries`. See tweag/rules_nixpkgs#417
Using go 1.21 fails with `GOPROXY list is not the empty string, but contains no entries`. See tweag/rules_nixpkgs#417
Using go 1.21 fails with `GOPROXY list is not the empty string, but contains no entries`. See tweag/rules_nixpkgs#417
Using go 1.21 fails with `GOPROXY list is not the empty string, but contains no entries`. See tweag/rules_nixpkgs#417
Newer go versions need at least rules_nixpkgs 0.11, see tweag/rules_nixpkgs#417.
Go 1.21 added a requirement for a 'go.env' file. Without that file, Go will throw an error like:
This commit adds it to the SDK exposed by rules_nixpkgs so that it is compatible with rules_go.
See: