-
Notifications
You must be signed in to change notification settings - Fork 252
Description
Hi,
I'm trying to upgrade [email protected] to a higher version of the library and I'm facing some troubles with grunt recognising the flag report-directory. To debug this issue I've used this simple grunt task, that prints all the flags that are being passed:
'use strict';
module.exports = (grunt) => {
grunt.registerTask('grunt_options', 'Runs grunt options debugging', function run() {
console.log('Those are all the grunt options: ', grunt.option.flags());
});
};
With some trial and error, I've been able to track down where this regression seems to have happened:
- Run
npm i [email protected] --save-dev
- Execute grunt task
node/node node_modules/.bin/grunt grunt_options --report-directory=1 --report-dy=2 --report-wd-directory=3
- Uninstall [email protected] and install 1.4.0
npm uninstall grunt && npm i [email protected]
- Run the task in step 2
For [email protected]
the output of the task is:
Those are all the grunt options: [
'--report-directory=1',
'--report-dy=2',
'--report-wd-directory=3',
'--gruntfile=/Users/roguib/dev/visual-builder/mcs_buf/breeze/vbcs-client/Gruntfile.js'
]
However, for [email protected]
the output no longer includes --report-directory
flag:
Running "grunt_options" task
Those are all the grunt options: [
'--report-dy=2',
'--report-wd-directory=3',
'--no-respawning',
'--gruntfile=/Users/roguib/dev/visual-builder/mcs_buf/breeze/vbcs-client/Gruntfile.js'
]
I've noticed by looking at the diff v1.3.0...v1.4.0 it could be caused by a regression in grunt-cli
library. Adding an override to grunt-cli
to ~1.3.2
instead of the ~1.4.2
version that comes with the new release seems to solve the issue:
"overrides": {
"grunt": {
"grunt-cli": "~1.3.2"
}
},
In that case, the task correctly identifies --report-directory
flag:
Those are all the grunt options: [
'--report-directory=1',
'--report-dy=2',
'--report-wd-directory=3',
'--gruntfile=/Users/roguib/dev/visual-builder/mcs_buf/breeze/vbcs-client/Gruntfile.js'
]
This problem is also reproducible with the latest release of [email protected]
and grunt-cli@~1.4.3
.