Re: Mojo Descriptor ... outdated documentation?

2021-09-22 Thread Tamás Cservenák
Howdy, requiresReports is unsupported since Maven 3, unsure if I would add it doco at all (rather would remove it). Thanks T On Wed, Sep 22, 2021 at 9:15 PM Slawomir Jaranowski wrote: > Hi, > > We have documentation about the Mojo descriptor and plugin api... [1] > > Its look like params: > >

Mojo Descriptor ... outdated documentation?

2021-09-22 Thread Slawomir Jaranowski
Hi, We have documentation about the Mojo descriptor and plugin api... [1] Its look like params: - executionStrategy - requiresDirectInvocation - inheritByDefault (PluginDescriptorBuilder - use only inheritedByDefault) - requiresReports are not used currently, I was looking in 3.8.x and mas

[GitHub] [maven-artifact-transfer] cstamas merged pull request #24: [MSHARED-987] Make use of SISU

2021-09-22 Thread GitBox
cstamas merged pull request #24: URL: https://github.com/apache/maven-artifact-transfer/pull/24 -- 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: de

[GitHub] [maven-artifact-transfer] michael-o commented on pull request #24: [MSHARED-987] Make use of SISU

2021-09-22 Thread GitBox
michael-o commented on pull request #24: URL: https://github.com/apache/maven-artifact-transfer/pull/24#issuecomment-924734705 @cstamas Go ahead a merge it. I appreciate your high quality work. -- This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [maven-artifact-transfer] cstamas commented on pull request #24: [MSHARED-987] Make use of SISU

2021-09-22 Thread GitBox
cstamas commented on pull request #24: URL: https://github.com/apache/maven-artifact-transfer/pull/24#issuecomment-924643199 > There are so many changes that I will blindly trust you here. Agreed, that's why I want to move forward, as IMHO people will realize how _simpler_ this will