gulp-cli icon indicating copy to clipboard operation
gulp-cli copied to clipboard

Remove references to description configuration property

Open folliehiyuki opened this issue 9 months ago • 4 comments

The description option in .gulp.* configuration file is never read.

folliehiyuki avatar Mar 22 '25 08:03 folliehiyuki

It looks like I never removed the docs for the description property when converting everything to the translation code. I think we'll probably just remove the docs since we want all of the message logic to go through the translation mechanism.

Do you want to update this PR to remove those docs?

phated avatar Jun 01 '25 20:06 phated

Is it just in the README? Some test files still use exports.description (https://github.com/gulpjs/gulp-cli/blob/master/test/fixtures/config/foo/bar/quux/.gulp.babel.js, https://github.com/gulpjs/gulp-cli/blob/master/test/fixtures/config/foo/bar/.gulp.js, https://github.com/gulpjs/gulp-cli/blob/master/test/fixtures/config/qux/.gulp.js) and they don't seem to be referenced anywhere. Do they need to be removed as well?

folliehiyuki avatar Jun 04 '25 17:06 folliehiyuki

I think we need to remove both the readme and unused test files. I forgot to remove both with my translation work.

phated avatar Jun 06 '25 14:06 phated

I deleted all the relevant files and npm run test still passed on my local machine.

folliehiyuki avatar Jun 06 '25 17:06 folliehiyuki