You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@maven.apache.org by GitBox <gi...@apache.org> on 2021/05/30 20:23:23 UTC

[GitHub] [maven-doxia] elharo opened a new pull request #65: replace deprecated class from commons-lang

elharo opened a new pull request #65:
URL: https://github.com/apache/maven-doxia/pull/65


   


-- 
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org


[GitHub] [maven-doxia] elharo merged pull request #65: replace deprecated class from commons-lang

Posted by GitBox <gi...@apache.org>.
elharo merged pull request #65:
URL: https://github.com/apache/maven-doxia/pull/65


   


-- 
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org


[GitHub] [maven-doxia] elharo commented on a change in pull request #65: replace deprecated class from commons-lang

Posted by GitBox <gi...@apache.org>.
elharo commented on a change in pull request #65:
URL: https://github.com/apache/maven-doxia/pull/65#discussion_r642658492



##########
File path: doxia-core/pom.xml
##########
@@ -59,6 +59,11 @@ under the License.
       <artifactId>commons-lang3</artifactId>
       <version>3.8.1</version>
     </dependency>
+    <dependency>
+      <groupId>org.apache.commons</groupId>
+      <artifactId>commons-text</artifactId>
+      <version>1.3</version>

Review comment:
       Because everything newer requires Java 8. :-(




-- 
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [maven-doxia] elilja commented on a change in pull request #65: replace deprecated class from commons-lang

Posted by GitBox <gi...@apache.org>.
elilja commented on a change in pull request #65:
URL: https://github.com/apache/maven-doxia/pull/65#discussion_r642982298



##########
File path: doxia-core/pom.xml
##########
@@ -59,6 +59,11 @@ under the License.
       <artifactId>commons-lang3</artifactId>
       <version>3.8.1</version>
     </dependency>
+    <dependency>
+      <groupId>org.apache.commons</groupId>
+      <artifactId>commons-text</artifactId>
+      <version>1.3</version>

Review comment:
       That's fine since some time ago, I believe. You may require Java 8 in plugins and components etc for the Maven eco system




-- 
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [maven-doxia] elharo commented on pull request #65: replace deprecated class from commons-lang

Posted by GitBox <gi...@apache.org>.
elharo commented on pull request #65:
URL: https://github.com/apache/maven-doxia/pull/65#issuecomment-851397741


   Build windows-jdk11 looks borked again :-(


-- 
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [maven-doxia] elilja commented on a change in pull request #65: replace deprecated class from commons-lang

Posted by GitBox <gi...@apache.org>.
elilja commented on a change in pull request #65:
URL: https://github.com/apache/maven-doxia/pull/65#discussion_r642508661



##########
File path: doxia-core/pom.xml
##########
@@ -59,6 +59,11 @@ under the License.
       <artifactId>commons-lang3</artifactId>
       <version>3.8.1</version>
     </dependency>
+    <dependency>
+      <groupId>org.apache.commons</groupId>
+      <artifactId>commons-text</artifactId>
+      <version>1.3</version>

Review comment:
       Why such an old version of commons-text?




-- 
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org