You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2022/10/20 18:39:46 UTC
[GitHub] [accumulo-examples] DomGarguilo opened a new pull request, #104: Bring some dependency versions up to date with main accumulo project
DomGarguilo opened a new pull request, #104:
URL: https://github.com/apache/accumulo-examples/pull/104
The following versions were bumped to match the main accumulo project dependency versions:
* parent version
* zookeeper
* sortpom-maven-plugin
* maven-enforcer-plugin
* maven-checkstyle-plugin
* puppycrawl
* impsort-maven-plugin
* formatter-maven-plugin
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [accumulo-examples] DomGarguilo commented on a diff in pull request #104: Bring some dependency versions up to date with main accumulo project
Posted by GitBox <gi...@apache.org>.
DomGarguilo commented on code in PR #104:
URL: https://github.com/apache/accumulo-examples/pull/104#discussion_r1001020066
##########
pom.xml:
##########
@@ -156,7 +156,7 @@
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-enforcer-plugin</artifactId>
- <version>3.0.0-M3</version>
+ <version>3.1.0</version>
Review Comment:
I'll try removing the entry entirely
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [accumulo-examples] DomGarguilo commented on a diff in pull request #104: Bring some dependency versions up to date with main accumulo project
Posted by GitBox <gi...@apache.org>.
DomGarguilo commented on code in PR #104:
URL: https://github.com/apache/accumulo-examples/pull/104#discussion_r1001030012
##########
pom.xml:
##########
@@ -156,7 +156,7 @@
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-enforcer-plugin</artifactId>
- <version>3.0.0-M3</version>
+ <version>3.1.0</version>
Review Comment:
Removed in 0cbfa26, enforcer runs with the correct version so should be good to go.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [accumulo-examples] DomGarguilo commented on a diff in pull request #104: Bring some dependency versions up to date with main accumulo project
Posted by GitBox <gi...@apache.org>.
DomGarguilo commented on code in PR #104:
URL: https://github.com/apache/accumulo-examples/pull/104#discussion_r1001019098
##########
pom.xml:
##########
@@ -156,7 +156,7 @@
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-enforcer-plugin</artifactId>
- <version>3.0.0-M3</version>
+ <version>3.1.0</version>
Review Comment:
I removed the version in ee1e9f7
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [accumulo-examples] DomGarguilo merged pull request #104: Bring some dependency versions up to date with main accumulo project
Posted by GitBox <gi...@apache.org>.
DomGarguilo merged PR #104:
URL: https://github.com/apache/accumulo-examples/pull/104
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [accumulo-examples] ctubbsii commented on a diff in pull request #104: Bring some dependency versions up to date with main accumulo project
Posted by GitBox <gi...@apache.org>.
ctubbsii commented on code in PR #104:
URL: https://github.com/apache/accumulo-examples/pull/104#discussion_r1001004177
##########
pom.xml:
##########
@@ -156,7 +156,7 @@
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-enforcer-plugin</artifactId>
- <version>3.0.0-M3</version>
+ <version>3.1.0</version>
Review Comment:
When you bump the parent POM, it might no longer be necessary to specify a version or have a dependencyManagement entry at all, for plugins whose version and config are identical (or older than) what's specified in the parent POM.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org