-
Notifications
You must be signed in to change notification settings - Fork 58
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
[MCHECKSTYLE-439] Upgrade to parent 40 #123
Conversation
To get rid of warnings. Also, downgrade plexus plugin from 2.1.1 to 2.1.0 as 2.1.1 is known to be broken. --- https://issues.apache.org/jira/browse/MCHECKSTYLE-439
whats wrong with plexus? |
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.
version for maven-invoker-plugin can be removed in line 360
Plexus 2.1.1 uses broken qdox, fixed here codehaus-plexus/plexus-containers@88f833b but never released (nor we want it released, as we should move off plexus not keep it updated) |
I can do a release anyway of the plexus container. We also made more cleanups |
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.
We can remove plexus annotation in next step
@cstamas Do you want to complete this? |
Go for it, am more on other topics currently |
To get rid of warnings.
Also, downgrade plexus plugin from 2.1.1 to 2.1.0 as 2.1.1 is known to be broken.
https://issues.apache.org/jira/browse/MCHECKSTYLE-439