-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Allow option to run recursive including root? #4906
Comments
I agree with you. And it used to work consistently in the past. But I was asked to change it: #2769 An option sounds good and I think such issue already exists but I can't find it. |
Hi @zkochan Thanks for the response, I had a look and found a special option in npm. https://docs.npmjs.com/cli/v8/commands/npm-link#include-workspace-root I believe using -r, --recursive flags should follow the |
close #4906 Co-authored-by: Zoltan Kochan <z@kochan.io>
Would it be possible to add this to the docs here: https://pnpm.io/filtering ? I'm very thankful the option exists, we have a CI where the root has its own lint script (for the root config files) |
According to https://pnpm.io/cli/recursive:
pnpm -r, --recursive
Runs a command in every project of a workspace, excluding the root project, when used with the following commands:
exec
run
test
add
Is there an option or is it possible to run with root as well? Or is that considered an anti-pattern?
Also, not sure why these specific commands have special hard-coded treatments? Is it part of some spec? I would have thought it would just be easier and more generic to have a flag say
--no-root
or something, and let the users decide if root should be included or not?The text was updated successfully, but these errors were encountered: