Looks like I gave the wrong command. Just tested this incantation and it worked on master.
mvn license:format -Dlicense.skip=false When I removed the license header on Drillbit.java it was added back again. ________________________________ From: Parth Chandra <par...@apache.org> Sent: Wednesday, April 18, 2018 3:54:36 PM To: dev Subject: Re: License Header FYI mvn license:format (not man license:format ) seems to not do anything. On Wed, Apr 18, 2018 at 3:53 PM, Parth Chandra <par...@apache.org> wrote: > man license:format does not seem t be doing anything. > Also, IMO it would be a good idea to enable rat checks by default. At the > very least we need to make sure that the license headers are there before > we check anything in. > > On Tue, Apr 17, 2018 at 1:41 PM, Timothy Farkas <tfar...@mapr.com> wrote: > >> Hi All, >> >> Recently the license formatting checks have become stricter and all the >> license headers have been reformatted. The main benefit from this is that >> it is no longer allowed to have license headers in java doc comments. This >> will help keep our javadocs clean when we publish them. By default license >> checks are disabled, but they are enabled for Travis. To manually enable >> license checks locally add -Drat.skip-false and -Dlicense.skip=false args >> to your maven command. Also to automatically add license headers to your >> new files do mvn license:format >> >> For the next couple days please manually check the license headers for >> your PRs. Also if you regenerate classes in drill/protocol please manually >> run mvn license:format to add the license headers. This is necessary >> because my last change broke Travis and auto formatting of licenses for >> generated classes, but this will be fixed after >> https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_apache_drill_pull_1215&d=DwIBaQ&c=cskdkSMqhcnjZxdQVpwTXg&r=4eQVr8zB8ZBff-yxTimdOQ&m=vnpyMg6WPxgAajLqWgn0FRdxqUb1IQJObkIOKqh-MOE&s=guKPyopeVkavMDrswmmScd2mXR_ZQvdwjG2oyy02M9U&e= >> is merged. >> >> Thanks, >> Tim >> > >
