-
Notifications
You must be signed in to change notification settings - Fork 6
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 from tools.deps.alpha #10
Comments
Hey, happy to hear you've enjoyed the videos! To be honest I'm not sure, until someone does a deep dive to compare the two approaches it's hard to say how we could benefit from the work there. If I had to guess I'd say we're a bit more aggressive in setting up or replacing classloaders, and we have logic to make sure upgraded dependencies get precedence over the old versions. I wouldn't expect a core library to do that kind of shenanigans. So unless someone can make a compelling technical case I think we default to doing nothing. The current approach will continue to work. The nature of this classpath stuff is that it's not too hard to get working in general, but there are almost always edge cases in particular when interacting with other tools and libraries, so I think it's good to have multiple options with different approaches. People can pick what's must appropriate for them. |
That said a simple version bump would be fine, if that does break things then it's better to find out. |
Hi @patbrown, no big rush, but would you be interested in this PR? |
I don't know exactly what the deal is with tools.deps and tools.deps.alpha. My knowledge of their inner-workings is limited. But classpath is reliant on
create-basis
slurp-deps
from tools.deps.alpha. Do you want to upgrade? If so, would you like a PR?I only ask because I really have no idea what a prudent plan around transitioning looks like. Both functions are present and the PR wouldn't take me much time, but if updating breaks other things you've got going on or your users I don't want to bother with it. I'm using both deps and deps.alpha via a hacky deps.edn trick for the time being.
Throughout the years I've enjoyed learning with your videos, episode 32 really changed how I treated metadata. And Classpath is an important tool in my dev workflow, so... THANKS FOR THAT!
The text was updated successfully, but these errors were encountered: