-
-
Notifications
You must be signed in to change notification settings - Fork 997
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
proposal: drop support for older go versions #963
Comments
Good points, I agree. I'd vote to drop support for Go 1.17 and older. Go 1.18 would be enough to replace the deprecated |
This is actually a great idea. Even if we go for "latest+4" to be more conservative, we'd still support Go 1.19 as of today (and Go 1.20 in Feb '25), which should be fine for most of the users. 👍 |
I think we should go a little bit past EOL cutoff, because there are some environments that are slow to adopt new version of go and even slower to remove support for older ones, most notably Google App Engine. Latest +3 or +4 sounds like a reasonable compromise. |
Just FTR, it seems app engine isn't that far from the upstream support timeline. Latest 1.23 is still in preview, but 1.20 has already been EOLd: And just as a devil's advocate, I see two more options:
But in summary, yes, I'd be totally ok with just +4 overall as well. |
good discussion. I think latest +4 is a good place to be in general. I’d say for short term if we broke the rule to +3 so we could bump up to 1.20 given that 1.24 is coming out soon and we’d then be at latest+4. And for any Go versions older, they can use an older tag of chi. |
Chi currently supports go down to 1.14, which has been EOL for almost 4 years.
This means keeping "code smells" around (e.g.
ioutil
) and blocking access to some newer features (e.g.ResponseController
).I propose bumping the minimal version to allow for more aggressive deprecations and feature adoption.
Concretely, I'd propose matching go's support policy of "latest+2", but would be happy with any other such dynamic approach, so that there's
WDYT?
The text was updated successfully, but these errors were encountered: