mabrarov commented on PR #148:
URL: 
https://github.com/apache/maven-assembly-plugin/pull/148#issuecomment-1624231912

   Hi @elharo,
   
   Regarding that 
[comment](https://github.com/apache/maven-assembly-plugin/pull/148#pullrequestreview-1514449650):
   
   1. Integration test for the new configuration option was added.
   1. The case when invalid value is specified for `overrideUmask` plugin 
configuration option is described in Javadoc for 
`org.apache.maven.plugins.assembly.mojos.AbstractAssemblyMojo#overrideUmask` 
field. This Javadoc is used for generation of plugin documentation - plugin.xml 
- so I find it sufficient.
   
   Thank you for your help with review of this pull request.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to