-
Notifications
You must be signed in to change notification settings - Fork 34
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
Update to clojure 1.10.3 and tools deps 1.10.3.814 #105
Update to clojure 1.10.3 and tools deps 1.10.3.814 #105
Conversation
Thanks for the PR @imrekoszo! We only update to stable releases of tools-deps. The latest of which is currently 1.10.2.796 (see here: https://github.com/clojure/brew-install/blob/1.10.3/CHANGELOG.md). And you can pull in any version of Clojure itself in your apps running inside the container. Once there is a stable release of the 1.10.3.x series we'll put out an update with it. |
Thanks @cap10morgan, I didn't realize that was still marked as prerelease. Perhaps the link you kindly posted is out of sync with https://clojure.org/releases/tools#_releases ? I asked Alex Miller about it on Slack, waiting for his clarification now. |
@imrekoszo Ah, yeah, looks like there might be some out-of-sync-ness going on there. I bet you're right that that is intended to be a stable release after all. If Alex confirms I'll reopen this PR. |
@cap10morgan thank you, that should now be updated. Alex also mentioned there are a few other places to check whether a release os stable https://clojurians.slack.com/archives/C06MAR553/p1616163064127800?thread_ts=1615911301.014400&cid=C06MAR553 |
Thanks @imrekoszo. I'll review this shortly. |
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.
Looks good to me! Thanks again @imrekoszo!
Official Docker images PR: docker-library/official-images#9822 |
Thank you very much! |
This looked straightforward enough so I went ahead with it.