-
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
Use maven-plugin-plugin in version managed in parent #97
Conversation
May I ask for review, please? |
makes sense! a release of this mojo-parent is planned soon, i think it would be best to update to this next version once released and remove the plugin version in the pom of this plugin. |
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.
as alternative we could just remove the plugin version immediately, because the mojo-parent 50 that is currently references already contains maven-plugin-plugin 3.5.1 - we forgot to remove the local plugin version when updating to parent 50.
Currently used version generates invalid javadoc for `target/generated-sources/plugin/org/codehaus/mojo/buildhelper/HelpMojo.java`: ```java 346 /** 347 * Splits the specified text into lines of convenient display length. 348 * 349 * @param text The text to split into lines, must not be <code>null</code>. 350 * @param indent The base indentation level of each line, must not be negative. 351 * @param indentSize The size of each indentation, must not be negative. 352 * @param lineLength The length of the line, must not be negative. 353 * @return The sequence of display lines, never <code>null</code>. 354 * @throws NegativeArraySizeException if <code>indent < 0</code> 355 */ 356 private static List<String> toLines( String text, int indent, int indentSize, int lineLength ) ``` where line 354 contains `<` instead of entity. 3.5.1 m-p-p has it fixed (MPLUGIN-324) and comes from current parent.
Hmm... it seems I didn't push corrected version. |
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.
lgtm
Currently used version generates invalid javadoc for
target/generated-sources/plugin/org/codehaus/mojo/buildhelper/HelpMojo.java
:where line 354 contains
<
instead of entity.3.5.1 m-p-p has it fixed (MPLUGIN-324) and comes from current parent.