[GitHub] [maven] rfscholte commented on pull request #394: [MNG-7020] Remove Maven 2 WagonExcluder backward compat code

2021-10-04 Thread GitBox 


rfscholte commented on pull request #394:
URL: https://github.com/apache/maven/pull/394#issuecomment-933213272


   @cstamas @michael-o I thought we also worked on that "require 
maven-plugin-api version", but can't find the ticket. For me that must be 
implemented first before accepting this PR.


-- 
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: issues-unsubscr...@maven.apache.org

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

[GitHub] [maven] rfscholte commented on pull request #394: [MNG-7020] Remove Maven 2 WagonExcluder backward compat code

2021-02-05 Thread GitBox 


rfscholte commented on pull request #394:
URL: https://github.com/apache/maven/pull/394#issuecomment-773483832


   IIRC this triggered a new issue. I'll remove my approval and need to have 
another look at it.



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:
us...@infra.apache.org

[GitHub] [maven] rfscholte commented on pull request #394: [MNG-7020] Remove Maven 2 WagonExcluder backward compat code

2021-02-04 Thread GitBox 


rfscholte commented on pull request #394:
URL: https://github.com/apache/maven/pull/394#issuecomment-773483832


   IIRC this triggered a new issue. I'll remove my approval and need to have 
another look at it.



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:
us...@infra.apache.org