search

open-cli-tools / concurrently

Run commands concurrently. Like `npm run watch-js & npm run watch-less` but better.
https://www.npmjs.com/package/concurrently
MIT License
7k stars 228 forks source link

Upgrade to pnpm 8 #410

Closed paescuj closed 10 months ago

paescuj commented 1 year ago

Postponed to EOL of Node.js 14 (2023-04-30)

danny007in commented 1 year ago

better to use only latest npm is there any special use of pnpm

paescuj commented 1 year ago

better to use only latest npm is there any special use of pnpm

What do you mean exactly?

danny007in commented 1 year ago

mean exactly?

Better to replace pnpm to latest npm

gustavohenke commented 1 year ago

Hey @danny007in, why do you think so?

danny007in commented 1 year ago

Hey @danny007in, why do you think so?

Exactly that is my point, using stable npm package best option, it also links files from the global store.

paescuj commented 1 year ago

[...] it also links files from the global store.

Did I miss something here? I'm not aware of this. Could you share a link?

Besides that, I don't think we're going back to npm (at least for now). It has been shown that pnpm works very well here. In particular, it has helped to greatly reduce the duration of the tests.

danny007in commented 1 year ago

[...] it also links files from the global store.

Did I miss something here? I'm not aware of this. Could you share a link?

Besides that, I don't think we're going back to npm (at least for now). It has been shown that pnpm works very well here. In particular, it has helped to greatly reduce the duration of the tests.

npm v9 works as pnpm now.

https://stackoverflow.com/questions/74569635/npm-9-doesnt-copy-js-files-for-local-dependencies

coveralls commented 11 months ago

Coverage Status

coverage: 99.197%. remained the same when pulling c95a841f95440519687b0b2e879030c84ff6fc92 on paescuj:pnpm-8 into 77fbc03b1a608022f449af90d6beca6f41e53614 on open-cli-tools:main.