-
Notifications
You must be signed in to change notification settings - Fork 82
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
Add skip flags #186
Add skip flags #186
Conversation
ad8a940
to
4f3495c
Compare
src/main/java/org/codehaus/mojo/buildhelper/AbstractAddResourceMojo.java
Outdated
Show resolved
Hide resolved
src/main/java/org/codehaus/mojo/buildhelper/AbstractAddResourceMojo.java
Outdated
Show resolved
Hide resolved
There is an issue #153 |
There is also We can think if we want one name We can also have two options - one global skip for all goals and specific one for each goals. |
src/main/java/org/codehaus/mojo/buildhelper/AddResourceMojo.java
Outdated
Show resolved
Hide resolved
You could just merge the PR(s) and then make the changes that you feel necessary yourself.
This is not an academic grading exercise where I am trying to get to a passing grade or something to be "acceptable". It is a contribution of a useful feature to a project that I did in my spare time trying to be helpful to the project. I already spent more time on this than I wanted. I am not spending more time. If you feel that this is not "production ready" code, then you need someone that gets paid for it to get it to that level. |
@hgschmie tanks for your works ... I don't to examine you or offend you in any way. I will care about this PR. |
- build-helper:add-source - build-helper:add-test-source - build-helper:add-resource - build-helper:add-test-resource Add `<skip>true|false</skip>` to allow skipping plugin execution.
Align with uptodate property / uptodate properties
5147fbd
to
c98b85e
Compare
c98b85e
to
2d63c03
Compare
Add
<skip>true|false</skip>
to allow skipping plugin execution.