make manifestversiondetect config default to true#1410
make manifestversiondetect config default to true#1410KristofferC wants to merge 1 commit intoJuliaLang:mainfrom
Conversation
|
I support defaulting to true after having used it for a while. |
|
I think we might want to notify of this behavior change when juliaup updates itself, which is currently silent. |
|
Proposal for that notification system here #1411 |
I'm not sure I would want that. You need to recompile your environment, etc. Part of the niceness of this is, imo, that things kind of keep working the way they did. And if you want to force a specific julia version, you can use |
From the priority list I think people mostly rely on the last one, the default channel. This comes above the default channel, so it does change things. The default channel tends to move with the patch releases and not keep older patch releases installed. I expect people will start running into this a lot: So we'd suggest them to install and hence keep using an older patch release, even though in practice we'd recommend I appreciate that the current approach is simpler, but I'm afraid it will cause people to run behind on julia patches, and/or use different patch releases at the same time without intending to. But perhaps that is the difficulty of equating the julia version the manifest last got resolved with, with the version that we want people to use by default. |
|
How about this: when Juliaup uses a Project/Manifest for Julia selection, it also reads the If the user picks If the update would amount to a minor version jump, we could also offer options that fix the compat to minor version upgrades etc. |
|
In terms of changing the default, we should wait a little bit. I believe we never shipped a version of Juliaup to any Windows user that had the feature at all (because we temporarily lost access to the Windows Store listing...), so ideally we would test it for a while with Windows users before we switch things over. |
|
We probably also need to add better support in the VS Code extension before we ship that, right now I got some very cryptic crashes with this option enabled. |
This is working quite nicely for me at least.