-
Notifications
You must be signed in to change notification settings - Fork 1
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 go1.21, use for go1.21.0 #522
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.
The change looks good, but CI is failing with curl: (6) Could not resolve host: null
even on retry - an unrelated issue, but strange nonetheless?
Oh, it seemed unrelated because there were a dozen error lines (so it seemed multiple assets were failing the same way), but I guess the additional lines were just retries, so they were all from the newly added assets - whose URLs have just been fixed in the followup commits? |
I think this has to do with an upstream artifact naming change. Traditionally, the first release doesn't have a |
It looks like the |
This adds go1.21.0, and uses go1.21.0 for go1.21. It also adds a simple test for the go1.21 line.
Supersedes #521.