Skip to content
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

grunt.util.spawn throwing empty error object #12

Open
wants to merge 82 commits into
base: master
Choose a base branch
from

Conversation

pearsop
Copy link

@pearsop pearsop commented Aug 22, 2017

branch: issue#70
file: tasks/index.js
line number: 112
description: need to change the if statement to check for an empty object b/c grunt.util.spawn throws an empty error object if successful.

pocesar and others added 30 commits July 1, 2013 09:16
Support mocha --recursive
…: restricts coverage to another set of files than "."
added mochOption parameter.
The official keyword for grunt plugins is gruntplugin, this is how it can be searched for and how it'll show up on gruntjs.org (it doesn't now).
Also, you can now specify options.require as either an array or a string.
nicokaiser and others added 30 commits April 17, 2015 10:36
When this task is run with a different node.js version than the "node"
executable in the path (or even worse, if no "node" is in the path), we have
a version mismatch, which leads to strange effects if we are dealing with
compiled modules.

So it is better to use `process.execPath` which should point to the
currently running version of node.
- friendly error messages if packages are not installed
- packages checked upon task execution; not before
- `README.md` updated to reflect changes; assuming imminent bump to v3.0.0; fixed Mocha site URL

Tangential:

- modify `Gruntfile.js` to run the actual test(s), since I didn't want to send a PR w/o running them
- add a `test` target to `package.json`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.