Hi, Given the priject is not an ASF one why would it have an ASF header?
Thanks, Justin On Tue, 19 Nov 2019, 19:52 Christofer Dutz, <christofer.d...@c-ware.de> wrote: > 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" <jus...@classsoftware.com>: > > 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 > > > >
