midniteio / multi-cuke

Parallel cucumber-js testing via Node's child_process.spawn api
13 stars 4 forks source link

Removing upper limit max-workers #37

Closed midniteio closed 8 years ago

midniteio commented 8 years ago

This is in reference to issue https://github.com/midniteio/multi-cuke/issues/35

Forcing hard limit to max-workers via cpus available is not strictly required and should not be enforced. Defaults are already tied to number of cores, and the value passed should be the explicit value used (this puts more responsibility on the caller of multi-cuke to know limits, but doesn't artificially restrict them to one browser per core).

@jimusbobus for reference.