diff options
author | Timothy Farkas <timothyfarkas@apache.org> | 2018-03-15 14:27:55 -0700 |
---|---|---|
committer | Aman Sinha <asinha@maprtech.com> | 2018-05-11 14:56:07 -0700 |
commit | dc8d0107cb6a4a31e1beaa9c98d6e51cf7486a67 (patch) | |
tree | 2457e58b5e5210c6d52c827c3ce6e8ad5ee8e48a /docs/dev/LicenseHeaders.md | |
parent | 8a1a7c53fb211dcba6e7b2f2ce90c28af4b9c518 (diff) |
DRILL-6249: Adding more unit testing documentation.
close apache/drill#1251
Diffstat (limited to 'docs/dev/LicenseHeaders.md')
-rw-r--r-- | docs/dev/LicenseHeaders.md | 34 |
1 files changed, 34 insertions, 0 deletions
diff --git a/docs/dev/LicenseHeaders.md b/docs/dev/LicenseHeaders.md new file mode 100644 index 000000000..21634edde --- /dev/null +++ b/docs/dev/LicenseHeaders.md @@ -0,0 +1,34 @@ +# License Headers + +Drill uses two license header checkers: + +* [Apache RAT Plugin](http://creadur.apache.org/rat/apache-rat-plugin/) +* [License Maven Plugin](http://code.mycila.com/license-maven-plugin/) + +## Why Two? + +[Apache RAT Plugin](http://creadur.apache.org/rat/apache-rat-plugin/) is used because it is the standard license header +checker for Apache projects. + +[License Maven Plugin](http://code.mycila.com/license-maven-plugin/) performs stricter license checks and supports disallowing license headers wrapped in `/**` and `**/`. This +allows us to inforce requiring all license headers to be wrapped only in `/*` and `*/`. + +## Doing License Checks + +The license checks are disabled locally by default and are enabled on Travis. If you'd like to perform +license checks locally you can do the following: + +``` + mvn license:check -Dlicense.skip=false +``` + +## Auto Formatting Headers + +If the license checks fail and you can't figure out what's wrong with your headers, you can auto-format +your license headers with the following command: + +``` +mvn license:format -Dlicense.skip=false +``` + +This command will also add license headers to files without them. |