You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by "cstamas (via GitHub)" <gi...@apache.org> on 2024/01/10 17:14:00 UTC

[PR] [MNG-7854] Warn if imported dep is ignored [maven]

cstamas opened a new pull request, #1370:
URL: https://github.com/apache/maven/pull/1370

   Resurrected old PR. Needs something like https://issues.apache.org/jira/browse/MNG-8004 
   
   ---
   
   https://issues.apache.org/jira/browse/MNG-7854
   


-- 
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-unsubscribe@maven.apache.org

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


Re: [PR] [MNG-7854] Warn if imported dep is ignored [maven]

Posted by "cstamas (via GitHub)" <gi...@apache.org>.
cstamas merged PR #1370:
URL: https://github.com/apache/maven/pull/1370


-- 
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-unsubscribe@maven.apache.org

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


Re: [PR] [MNG-7854] Warn if imported dep is ignored [maven]

Posted by "gnodet (via GitHub)" <gi...@apache.org>.
gnodet commented on PR #1370:
URL: https://github.com/apache/maven/pull/1370#issuecomment-1921029749

   I've added a few unit tests.  I've slightly changed things so that the warnings are only created when the dependency is not directly managed.  The warning also suggests to add a direct managed dependency to get rid of it, so it's now helpful and actionnable.
   
   That's clearly a good first step.  The next one could be to use a "closest wins" strategy instead of this "direct wins, then first wins" weird ad-hoc strategy.


-- 
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-unsubscribe@maven.apache.org

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