-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
wip: treat links with installLinks set as Links not Nodes #5820
base: latest
Are you sure you want to change the base?
Conversation
8dbff45
to
1192d89
Compare
@lukekarrys, I honestly hate to be this person but I was wondering if there's anything we can do to help move this forward? We're stuck on npm 8 because of this issue but would love to start using some neat features that became available in v9. We have about 15 developers working on 40+ repositories so the workaround mentioned in in #5844 isn't really working out for us. Again, no pressure, we're just interested in an update since it's been a few months ❤️ |
@cascornelissen Thanks for reaching out and offering help. Unfortunately, this ended up being a tricky bug to fix and the proposed WIP fix in this PR ended up causing other bugs. We still consider this
And to follow up on this specific, the commands I wrote for that workaround were targeted to individual projects but it should be possible to get the end result at scale. The end goal is for each project to have a root level
install-links=false Apologies if you've already tried that, but if you really want to use |
Thanks for the update @lukekarrys, good to hear this issue wasn't lost somehow. With regards to the We'll stick with |
No description provided.