node < 4 support PR?

Would you accept a pr which utilizes webpack + babel to expose an es5 version?

Currently the line lower versions of node trip up on lib/index.js:139

var err = {
  code: code,
  message: '`' + commandStr + '` failed with code ' + code,
  childProcess: cp,
  toString() { // <------------- invalid syntax
    return this.message;
  }
};

You could keep the default require pointing to your current entry point, and have webpack output a file es5.js which would allow people to require('child-process-promise/es5').

As for “why?” - my corporate environment runs build servers using node v0.12.14 and so our development environment should run on the same. Getting my co-workers using the command line is hard enough, but if they have to remember to mess with nvm in order to use the few CLI tools I create, that’s a hurdle they shouldn’t have to deal with.

I appreciate the module regardless!

[olsonpm]

Working on it now and will create a PR. I should have closed this issue when we resolved the direction anyway.

[olsonpm]

Interesting - yeah I thought there were non-browser use-cases for bundlers, but the more I google the more I find “how to bundle for the server environment” instead of “why”. I’ll use the conditional as you explained.

Thanks for mentioning lasso - hadn’t heard of it before.