VERIFY_ONLY should verify all files before failing

Question

VERIFY_ONLY should verify all files before failing

tomfuertes opened this issue 11 years ago · comments

When I have 1+ files out of verification, it'd be nice if both were listed before failing in a non-forced run. Example w/ comments below.

$ grunt jsbeautifier:verify # Fails on first file
Running "jsbeautifier:verify" (jsbeautifier) task
Warning: app/assets/javascripts/CSVConvertor.js was not beautified Use --force to continue.

Aborted due to warnings.

$ grunt jsbeautifier:verify --force # Prints everything but doesn't fail
Running "jsbeautifier:verify" (jsbeautifier) task
Warning: app/assets/javascripts/CSVConvertor.js was not beautified Used --force, continuing.
Warning: app/assets/javascripts/ab_testing.js was not beautified Used --force, continuing.
Beautified 37 files, changed 0 files...OK

Done, but with warnings.

<3 -Tom

Derrick Pelletier · Answer 1 · Sat Dec 14 2013 03:05:40 GMT+0800 (China Standard Time)

Agreed. I find the VERIFY_ONLY functionality to be confusing for this same reason. +1

Vishal Kadam · Answer 2 · Sun Dec 15 2013 02:15:27 GMT+0800 (China Standard Time)

Fixed in 0.2.5