-
Notifications
You must be signed in to change notification settings - Fork 170
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
How do we run upgrade global package managers #395
Comments
What's the difference from |
|
I disagree this is a behavior we want to have. Package managers don't have this concept of "upgrade but stay on the same major", and I don't think we should add a whole new command just for that. Some alternatives:
|
Why do we have a |
I think I implemented the "same major restriction" there because someone else requested it and semantically it made some sense to be conservative for a project with a checked-in version. I don't think it makes that much sense for the global version - if someone wants to use a specific version somewhere, they should lock it with |
I just want to have one command that upgrades Yarn to the latest 1 and everything else to the latest major. It was removed in #351 with no documentation. |
Corepack is still experimental, so a regression in that way isn't necessarily to be fixed. Especially the part about "Yarn to the latest 1 and everything else to the latest major" is worth a debate. The one and only reason Since Corepack isn't used by those older projects, it doesn't have the same concerns, so it's worth really considering whether it makes sense to keep yarn@1 the default global there. |
If I install Yarn modern globally, will Yarn classic still be used if I don't have |
In my experience, most folks want to be up-to-date, but would prefer staying away from breaking changes, so I really think it makes sense to have.
I agree it's debatable, but having this feature doesn't force anyone to use it – same way the fact that Corepack supports the |
I'm trying this and it's not working. As soon as Yarn is upgraded globally with It seems if I don't want to upgrade legacy Yarn projects while using corepack, I have to either only update Yarn locally or explicitly set For now I'm avoiding upgrading package managers globally with corepack to improve compatibility. For security purposes, I'll still update my global npm and local Yarn/pnpm versions. |
I am not sure how this should work, however, in my case, While the log says Is there an additional command I need to run so that PNPM get upgraded to its latest version and made the default version to be used in all new shells/terminals? Moreover, even The only thing that actually works is Thanks! 🙏 |
@tukusejssirs What's your environment? I've seen this behavior in Github Codespaces. Is that where you're trying to do this? |
@Tobbe, I use Linux (specifically Fedora with SwayWM). And I don’t use GitHub much (apart from contributions), as I prefer GitLab. So, this happens on my local machine. |
@tukusejssirs What does Also: What's your node version? The issue actually seems to be fixed for Github Codespaces, and I noticed that when I last tried they were on Node 20.14.0, now they're on 20.15.1. Someone else also reported a similar issue on Node 20.11.0. So maybe try upgrading Node if you aren't already on the latest version |
@Tobbe, it returns Note that I have installed Node.js and NPM via
I use After looking more into which Node version is used using As my installed In the end, the only Node version which does not work as expected, is Sorry for the noise. 😉 And thank you for helping me out! 🙏 PS—While my specific problem is solved, there still might be some issue somewhere between Corepack and NVM (I am not sure at which end). I presume I’ll have the same issue when I try to upgrade PNPM next time. 🤔 |
Yeah, switching around between different node version using nvm I noticed similar behavior (Github Codespaces are also using nvm). So there's definitely some weirdness going on between nvm and corepack |
Originally posted by @nickmccurdy in #351 (comment)
Anybody have thoughts on adding something like
corepack up -g yarn pnpm
?The text was updated successfully, but these errors were encountered: