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
If args['parallel'] is undefined, which it is when it’s not specified, then nxArgs['parallel'] is not set and defaults to undefined too.
I've tested this by adding this line directly below the snippet above:
output.note({
title: `Run with args.parallel being '${args['parallel']}', so nxArgs.parallel is set to '${nxArgs['parallel']}'`,
});
Running nx run-many --target=lint --all on my repository (or any nx repository for that matter) gives:
> NX NOTE Run with args.parallel being 'undefined', so nxArgs.parallel is set to 'undefined'
I can create a PR, but I need to know the Nx team's decision. Do you want this parallel value to default to 3 or do you want to update the documentation to match the current behaviour?
The text was updated successfully, but these errors were encountered:
Current Behavior
If the
--parallel
tag is not explicitly set, then it defaults toundefined
, although the documentation specifies that 3 is the default value.Expected Behavior
It should be defaulted to 3, or the documentation should be updated to say the default value is
undefined
, meaning sequential executionGithub Repo
https://github.com/nrwl/nx/blob/master/packages/nx/src/utils/command-line-utils.ts#L168-L180
Steps to Reproduce
nx run-many --target=lint --all
, without the--parallel
tag.Nx Report
Failure Logs
Additional Information
I believe the problem lies within this snippet:
https://github.com/nrwl/nx/blob/master/packages/nx/src/utils/command-line-utils.ts#L168-L180
If
args['parallel']
isundefined
, which it is when it’s not specified, thennxArgs['parallel']
is not set and defaults toundefined
too.I've tested this by adding this line directly below the snippet above:
Running
nx run-many --target=lint --all
on my repository (or any nx repository for that matter) gives:I can create a PR, but I need to know the Nx team's decision. Do you want this
parallel
value to default to 3 or do you want to update the documentation to match the current behaviour?The text was updated successfully, but these errors were encountered: