-
Notifications
You must be signed in to change notification settings - Fork 66
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
Modernize deps #185
base: master
Are you sure you want to change the base?
Modernize deps #185
Conversation
3990a14
to
8cf2819
Compare
to make understanding upgrades more convenient.
8cf2819
to
e7908d1
Compare
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.
Thanks for the effort, however it seems the CI jobs are broken, it is hard to tell what is the cause given the PR is quite large. A possible approach would be to open a smaller PR first to upgrade the test containers. And then see if upgrading the version packages will not introduce breakage.
|
||
"5:3" | ||
#"/throw1\.cljc:(\d+:\d+)" | ||
"5" |
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.
We lost the column information. What is the reason here?
:exclusions [rewrite-cljs/rewrite-cljs]}} | ||
{org.clojure/clojure | ||
{:mvn/version "1.11.1" | ||
:changelog "https://github.com/clojure/clojure/blob/master/changes.md"} |
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.
What are these changelog
fields? Is that something supported by tools.deps?
Keep the Clojure version usable on modern hardware (Apply Silicon aka M1 / M2 Macs).
Since, the CLJS variant relies on the unmaintained
lumo
project, this PR doesn't try to address dependency upgrades for that.