-
Notifications
You must be signed in to change notification settings - Fork 17.7k
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
x/tools/gopls: reenable tests suppressed due to lack of build{ssa,ir} support for range-over-func or gotypesalias=1 #67262
Comments
Change https://go.dev/cl/584395 mentions this issue: |
Until staticcheck supports range-over-func, gopls won't attempt to run it on packages that use range-over-func, which is about to be nearly all packages. Our existing tests will not work. So, disable them for now. Updates dominikh/go-tools#1494 Updates golang/go#67262 Change-Id: Ia498b6122cbb7d3797d5d552acbbf0b3dc736eb9 Reviewed-on: https://go-review.googlesource.com/c/tools/+/584395 LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com> Auto-Submit: Alan Donovan <adonovan@google.com> Reviewed-by: Robert Findley <rfindley@google.com>
Change https://go.dev/cl/584298 mentions this issue: |
I missed a test in CL 584395. Updates golang/go#67262 Change-Id: I9edbaf46cd148631925d7d70d64cfff4eb7ff660 Reviewed-on: https://go-review.googlesource.com/c/tools/+/584298 Reviewed-by: Tim King <taking@google.com> LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com> Auto-Submit: Alan Donovan <adonovan@google.com>
Current status:
|
Change https://go.dev/cl/623295 mentions this issue: |
@adonovan what's left after the above fixes? |
…er-func" This reverts commit a432b16 (CL 538778) Reason for revert: workaround no longer needed. Updates golang/go#67262 Change-Id: I995bca3a2d835880318ee2a68fa7352aa7ba8776 Reviewed-on: https://go-review.googlesource.com/c/tools/+/623295 Reviewed-by: Robert Findley <rfindley@google.com> LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com> Auto-Submit: Alan Donovan <adonovan@google.com>
Nothing, I think. |
A number of gopls tests have been skipped. We shouldn't forget to reenable them when the upstream issues are fixed.
Pending CL adding SSA support for range-over-func:
Upstream issues:
The text was updated successfully, but these errors were encountered: