[GitHub] [maven-dependency-plugin] HannesWell commented on pull request #257: [MDEP-674] Add m2e lifecycle metadata

[GitBox <gi...@apache.org>]

HannesWell commented on PR #257:
URL: https://github.com/apache/maven-dependency-plugin/pull/257#issuecomment-1300976071

   > @kwin @HannesWell what about adding the lifecycle mappings in https://github.com/eclipse-m2e/m2e-core/blob/master/org.eclipse.m2e.core/lifecycle-mapping-metadata.xml and not include it in the PR if this is an issue for Maven project? After all, @olamy is right and we have a way to put those mappings file somewhere else, only the BuildContext API has to be part of the plugin as far as I know.
   
   Yes we can do that. Actually it would be natural to have all metadata in the plugin to minimize the synchronization effort, but this does only make sense if that is in a general 'Maven' format and not a IDE specific one (as said above https://github.com/apache/maven-dependency-plugin/pull/257#discussion_r1012086763)


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