Skip to content
This repository has been archived by the owner on Aug 30, 2021. It is now read-only.

Rise gunt-concurrent task limit. #156

Merged
merged 1 commit into from
Oct 7, 2014
Merged

Rise gunt-concurrent task limit. #156

merged 1 commit into from
Oct 7, 2014

Conversation

miguelcoba
Copy link
Contributor

Fixes issue with default value for grunt-concurrent limit in machines with few cpu/cores.

From grunt-concurrent docs:

limit

Type: Number
Default: Number of CPU cores (require('os').cpus().length) with a minimum of 2

Limit of how many tasks that are run concurrently.

In machines with few cpus/cores, the default limit of 2 the 'grunt debug' task
can't start enough tasks for starting the node-inspector task, so the command fails
and the capp can't be debugged.

Fixes issue #144

Fixes issue with default value for grunt-concurrent limit in machines with few cpu/cores.

From grunt-concurrent docs:

	limit

	Type: Number
	Default: Number of CPU cores (require('os').cpus().length) with a minimum of 2

	Limit of how many tasks that are run concurrently.

In machines with few cpus/cores, the default limit of 2 the 'grunt debug' task
can't start enough tasks for starting the node-inspector task, so the command fails
and the capp can't be debugged.

Fixes issue #144
@miguelcoba
Copy link
Contributor Author

Resend pull request as the previous (#145) was from master and not from a topic branch.

amoshaviv added a commit that referenced this pull request Oct 7, 2014
Rise gunt-concurrent task limit.
@amoshaviv amoshaviv merged commit f5431da into meanjs:master Oct 7, 2014
@miguelcoba miguelcoba deleted the issue-144 branch October 7, 2014 14:58
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants