-
Notifications
You must be signed in to change notification settings - Fork 743
Closed
Description
Ran into an issue debugging an issue with a coworker.
If the string module is included anywhere downstream of shelljs you get a pair of warnings.
to: wrong arguments
toEnd: wrong argumentsHere is the script to duplicate the issue.
var shelljs = require('shelljs');
var S = require('string');
// will produce output...This has already been caught by several other issues...
- minor issue with shell.js jprichardson/string.js#73
- Error: to: wrong arguments paazmaya/grunt-trimtrailingspaces#1
- output: to: wrong arguments admc/wd#220
Since shelljs is so heavily relied upon it might be a better to not modify the global String prototype.