- Notifications
You must be signed in to change notification settings - Fork77
Only fail on error, add option to ignore warnings#120
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
This pull request extends on the error checking a bit to ignorewarnings. Warnings do not necessitate a complete failure. For example,with ES6, you're gonna get warnings about new stuff no matter what –you can still generate docs!Furthermore, added an ignoreWarnings option to not print warnings.
Isn't the role of the |
Yes, but the way this is being done makes no sense. It's failing immediately on warnings. Warnings in jsdoc and warnings in grunt are two different things. Most importantly, even if you --force, you don't see all the warnings and errors. jsdoc doesn't even die during errors, it keeps going and shows you all of them. I mostly just want all of the output, warnings included. |
simonomis commentedMar 20, 2015
This feature would be really useful for me too. I'm writing ES6 modules and because I have to use |
I'll merge it as soon as I have some time to do it. |
Only fail on error, add option to ignore warnings
Merged but I need to handle the |
This pull request extends on the error checking a bit to ignore
warnings. Warnings do not necessitate a complete failure. For example,
with ES6, you're gonna get warnings about new stuff no matter what –
you can still generate docs!
Furthermore, added an ignoreWarnings option to not print warnings.