Skip to content
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

[5.6.x]Update the version of the maven plugin #514 (#515) #517

Merged
merged 1 commit into from
Jan 20, 2021

Conversation

btootomoyuk
Copy link
Contributor

(cherry picked from commit bac8404)

Please review #516.

Confirmation:

  • I did a Grep search for plugin to see if there was any other place that managed the plugin version other than the place where I imported it with cherry-pick.

  • Check for new warnings or errors due to upgrading plug-ins.

    • maven-war-plugin : Confirmed that war generation succeeds without warning when mvn package is executed.

    • build-helper-maven-plugin : Confirmed that the source directory is added successfully without any warnings when the project is built.

    • maven-failsafe-plugin : Confirmed that the Test Result Report is generated without any warning when the test is executed.

    • maven-surefire-plugin : Configured, but not used in the project, so not tested.

@btootomoyuk btootomoyuk self-assigned this Jan 20, 2021
@btootomoyuk btootomoyuk changed the title Update the version of the maven plugin #514 (#515) [5.6.x]Update the version of the maven plugin #514 (#515) Jan 20, 2021
@btkatoufm btkatoufm merged commit 46c0173 into 5.6.x Jan 20, 2021
@btkatoufm btkatoufm deleted the issues/516_update_plugin branch January 20, 2021 23:42
btkatoufm added a commit that referenced this pull request Jan 21, 2021
btkobayashirun pushed a commit that referenced this pull request Jan 25, 2021
…nd update Maven War Plugin #520 (#518)

* Revert "Update the version of the maven plugin #514 (#515) (#517)"

This reverts commit 46c0173.

* update maven-war-plugin 3.3.1 #514 (#515) (#517), #520
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants