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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[FIXED #33] allow config to not fail the build on violations #34
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could use an IT I think.
* | ||
*/ | ||
@Parameter( defaultValue = "true", property = "animal.sniffer.error" ) | ||
protected boolean error; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
failOnError
?
} | ||
else | ||
{ | ||
getLog().info( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
warning
?
Agree all. Let me rework. |
The ant task should offer the same parameter. |
* fix javadoc issues in deprecated comments * update `animal-sniffer-ant-tasks` to 1.16 + patch to make it not fail the build (see mojohaus/animal-sniffer#34) * update the `check-plugins` task, make it not fail the build git-svn-id: https://josm.openstreetmap.de/svn/trunk@12873 0c6e7542-c601-0410-84e7-c038aed88b3b
* fix javadoc issues in deprecated comments * update `animal-sniffer-ant-tasks` to 1.16 + patch to make it not fail the build (see mojohaus/animal-sniffer#34) * update the `check-plugins` task, make it not fail the build git-svn-id: http://josm.openstreetmap.de/svn/trunk@12873 0c6e7542-c601-0410-84e7-c038aed88b3b
Allows a new configuration property 'error' which is true by default to maintain current behavior. The new property lets users not fail the build if errors are found, but simply to allow the error messages to be shown.