cziegeler commented on issue #13: SLING-8038 - Allow the Repository MOJO specify including features from CLI URL: https://github.com/apache/sling-slingfeature-maven-plugin/pull/13#issuecomment-431268901 It's a little bit hard to follow what exactly has changed. Maybe you could redo the patch without moving code out of the existing class first. We can later on then rename this class to AbstractXYZ The new mojo should not be bound to any lifecycle phase, otherwise it's not usable outside of a maven project. In addition the only parameter it should get are mvn coordinates for the feature, no includes, no directory etc. The mvn coordinates can be parsed with the existing code in ArtifactId
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on 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 With regards, Apache Git Services