Hi Justin,

this is a file I personally wrote and submitted to the maven-wrapper project. 
So technically it could have the Apache header ... after all, I put it there. 
So I think the Apache header doesn't have to be replaced.

Chris



Am 18.11.19, 21:21 schrieb "Justin Mclean" <[email protected]>:

    HI,
    
    I’m -1 on this as the maven wrapper [1] is still not being handled 
correctly, please read my previous comments. This is a 3rd party file and 
should not have an ASF header, you can't alter the header of a 3rd party file 
without permission. Thanks for fixing the other issues.
    
    Thanks,
    Justin
    
    1. .mvn/wrapper/MavenWrapperDownloader.java
    
    

Reply via email to