lbruun commented on issue #282: [NETBEANS-161] Changed license header introducer to simple slash-star URL: https://github.com/apache/incubator-netbeans/pull/282#issuecomment-345551323 I don't think it is any safer to use include list. After all the snippet only change those files which already today use `/**` as the beginning of the license header. The match must be at the beginning of the file. It checks the first 49 bytes of each file to see if there's a match. If anything the method includes too _little_, not too _much_, as it doesn't catch those Java files with CRLF which have inadvertently been committed to Apache NetBeans during the past 2 months. Examples: ```text autoupdate.services\test\unit\src\org\netbeans\modules\autoupdate\services\ExecutablePermissionsTestDisabled.java autoupdate.services\test\unit\src\org\netbeans\modules\autoupdate\services\InstallEagerModuleTest.java ``` (ok, these aren't examples, these are actually the only ones I've found. :-) )
---------------------------------------------------------------- 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