Tests failing on Go 1.22+ #312
Comments
benjaminjkraft
added a commit
that referenced
this issue
Feb 15, 2024
We weren't pinning the Go version for golangci-lint (only the golangci-lint version); seemingly it breaks on 1.21+. Now we're using a new version making it easy to pin back to 1.20. Later I'll look at why 1.21+ are failing (#313). While I was at it I added tests on 1.21 (1.22+ is failing, again will look later, #312), and upgraded the setup-go action itself. I have: - [x] Written a clear PR title and description (above) - [x] Signed the [Khan Academy CLA](https://www.khanacademy.org/r/cla) - [x] Added tests covering my changes, if applicable (n/a) - [x] Included a link to the issue fixed, if applicable (n/a) - [x] Included documentation, for new features (n/a) - [x] Added an entry to the changelog (n/a)
6 tasks
benjaminjkraft
added a commit
that referenced
this issue
Feb 18, 2024
I guess our x/tools was too old? I'm unclear on how x/tools versioning works with language versioning. Anyway, now it all works. Fixes #312. I have: - [x] Written a clear PR title and description (above) - [x] Signed the [Khan Academy CLA](https://www.khanacademy.org/r/cla) - [x] Added tests covering my changes, if applicable (n/a) - [x] Included a link to the issue fixed, if applicable - [x] Included documentation, for new features (n/a) - [x] Added an entry to the changelog
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Not sure what's up (possibly we need newer
go/packages? or ago/typesbug in 1.22?) but we'll probably need to fix it.The text was updated successfully, but these errors were encountered: