-
Notifications
You must be signed in to change notification settings - Fork 111
Plugin refactoring #450
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
base: master
Are you sure you want to change the base?
Plugin refactoring #450
Conversation
…eters; added BreakBuild and SkipReport classes.
…upport mojo testing.
… SkipModuleStrategy.
|
Thanks for submitting the pull request. Next week I am quite involved in other topics; hence I won't have enough time to review it properly. But after that week, I have put it on my todo list. |
|
Have you seen, the test are failing with |
|
I saw that. It's just a warning in my environment. I'll look into it. |
…uild error when building in a clean environment.
|
I think I solved the build problem. Do I need to do another pull request? |
|
Now all builds are green. Thanks for the updates. |
This branch contains a refactoring of the japicmp-maven-plugin. The main POM has been changed to use:
to facilitate proper unit tests for the plugin. The core plugin logic has been moved to:
and the japicmp-maven-plugin POM file has been enhanced to generate site reports for JaCoCo, Javadoc, etc.
The new unit tests increase code coverage to about 62%. I'll work on more tests to increase code coverage.