You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
As a developer and maintainer of a monorepo, I'd like to enforce that everyone adds a test:ci script to any new packages, as there's a PNPM command that we run: pnpm -r test:ci. However, this script will silently skip any packages that don't have that script.
Describe the solution you'd like
We would want PNPM to check recursively that the script exists. Right now, there's a --if-present flag (which is not required for recursive script, and is added implicitly). So, it would be great for there to be a --fail-if-not-present flag.
Describe the drawbacks of your solution
No drawbacks. Existing behavior would stay the same.
Describe alternatives you've considered
To my knowledge, no NPM packages that I can find do this kind of checking for a script being defined in all monorepo packages, and IMO this is more the responsibility of the script runner (in this case, PNPM).
The text was updated successfully, but these errors were encountered:
Describe the user story
As a developer and maintainer of a monorepo, I'd like to enforce that everyone adds a
test:ci
script to any new packages, as there's a PNPM command that we run:pnpm -r test:ci
. However, this script will silently skip any packages that don't have that script.Describe the solution you'd like
We would want PNPM to check recursively that the script exists. Right now, there's a
--if-present
flag (which is not required for recursive script, and is added implicitly). So, it would be great for there to be a--fail-if-not-present
flag.Describe the drawbacks of your solution
No drawbacks. Existing behavior would stay the same.
Describe alternatives you've considered
To my knowledge, no NPM packages that I can find do this kind of checking for a script being defined in all monorepo packages, and IMO this is more the responsibility of the script runner (in this case, PNPM).
The text was updated successfully, but these errors were encountered: