Zoltan Kochan
Zoltan Kochan
Overall, I am not opposed to this. However, this will also allow to set the value via commas in `pnpm-workspace.yaml`.
We'd have to change this for any flags supporting arrays. Otherwise, it would introduce inconsistency.
What if the value itself should contain a comma?
> Before executing a locally cached version pnpm dlx should revalidate that this is still the current version for that tag and fetch the new one if it is not....
I am not sure about this one. dlx is a unique command where the flags for pnpm CLI itself should be listed before the `dlx` keyword, so this works: ```...
fair point
This should already work if you run `pnpm publish`. pnpm replaces the catalogs in versions specs. Related test: https://github.com/pnpm/pnpm/blob/ce42d8fbbba03d01d1a809d1949f0c858e20133d/pkg-manifest/exportable-manifest/test/index.test.ts#L168
`shell-quote` was introduced in PR https://github.com/pnpm/pnpm/pull/9018 for fixing https://github.com/pnpm/pnpm/issues/8980 and https://github.com/pnpm/pnpm/issues/7641. We can revert that change.
I am not sure we want people to know about these. All these hooks except `filterLog` were added for easier integration with [stackblitz](https://stackblitz.com/). Would anyone else want to use these...
A few other companies are planning to use the hooks. I added short docs about them: https://github.com/pnpm/pnpm.io/commit/4e852ad5052cb3217560d72643522ce2bbea139c also, it is now easier to ship hooks via dependencies, using [configDependencies](https://pnpm.io/config-dependencies).