You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by GitBox <gi...@apache.org> on 2020/06/19 12:58:57 UTC

[GitHub] [maven-dependency-plugin] elharo opened a new pull request #60: [MDEP-645] update docs to reflect new target

elharo opened a new pull request #60:
URL: https://github.com/apache/maven-dependency-plugin/pull/60


   @jonvolfson
   
   


----------------------------------------------------------------
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-dependency-plugin] rfscholte commented on pull request #60: [MDEP-645] update docs to reflect new target

Posted by GitBox <gi...@apache.org>.
rfscholte commented on pull request #60:
URL: https://github.com/apache/maven-dependency-plugin/pull/60#issuecomment-666666503


   apache.org/foundation/glossary.html#CTR


----------------------------------------------------------------
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-dependency-plugin] elharo commented on a change in pull request #60: [MDEP-645] update docs to reflect new target

Posted by GitBox <gi...@apache.org>.
elharo commented on a change in pull request #60:
URL: https://github.com/apache/maven-dependency-plugin/pull/60#discussion_r443124446



##########
File path: src/site/apt/index.apt.vm
##########
@@ -72,6 +72,8 @@ ${project.name}
 
   *{{{./list-mojo.html}dependency:list}} alias for resolve that lists the dependencies for this project.
 
+  *{{{./list-classes-mojo.html}dependency:list-classes}} displays the fully package qualified names of all classes found in a specified dependency.

Review comment:
       damned if I know. Added hyphen. 




----------------------------------------------------------------
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-dependency-plugin] elharo commented on pull request #60: [MDEP-645] update docs to reflect new target

Posted by GitBox <gi...@apache.org>.
elharo commented on pull request #60:
URL: https://github.com/apache/maven-dependency-plugin/pull/60#issuecomment-666642720


   CTR?
   
   On Thu, Jul 30, 2020 at 2:00 PM Robert Scholte <no...@github.com>
   wrote:
   
   > Please switch to CTR and don't use PR for these kind of things, to me it
   > is causing way too much unnecessary noise.
   >
   > —
   > You are receiving this because you authored the thread.
   > Reply to this email directly, view it on GitHub
   > <https://github.com/apache/maven-dependency-plugin/pull/60#issuecomment-666566677>,
   > or unsubscribe
   > <https://github.com/notifications/unsubscribe-auth/AAHVP2HAS6QVOSRWHHAFVP3R6GYM7ANCNFSM4OCWRK2A>
   > .
   >
   
   
   -- 
   Elliotte Rusty Harold
   elharo@macfaq.com
   


----------------------------------------------------------------
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-dependency-plugin] rfscholte commented on pull request #60: [MDEP-645] update docs to reflect new target

Posted by GitBox <gi...@apache.org>.
rfscholte commented on pull request #60:
URL: https://github.com/apache/maven-dependency-plugin/pull/60#issuecomment-666566677


   Please switch to CTR and don't use PR for these kind of things, to me it is causing way too much unnecessary noise.


----------------------------------------------------------------
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-dependency-plugin] michael-o commented on a change in pull request #60: [MDEP-645] update docs to reflect new target

Posted by GitBox <gi...@apache.org>.
michael-o commented on a change in pull request #60:
URL: https://github.com/apache/maven-dependency-plugin/pull/60#discussion_r442944567



##########
File path: src/site/apt/index.apt.vm
##########
@@ -72,6 +72,8 @@ ${project.name}
 
   *{{{./list-mojo.html}dependency:list}} alias for resolve that lists the dependencies for this project.
 
+  *{{{./list-classes-mojo.html}dependency:list-classes}} displays the fully package qualified names of all classes found in a specified dependency.

Review comment:
       Shouldn't that read `package-qualified`?




----------------------------------------------------------------
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-dependency-plugin] elharo merged pull request #60: [MDEP-645] update docs to reflect new target

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


   


----------------------------------------------------------------
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-dependency-plugin] elharo commented on pull request #60: [MDEP-645] update docs to reflect new target

Posted by GitBox <gi...@apache.org>.
elharo commented on pull request #60:
URL: https://github.com/apache/maven-dependency-plugin/pull/60#issuecomment-668758909


   Ping @michael-o Since you requested changes on this, your approval is required


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