aboutsummaryrefslogtreecommitdiff
path: root/docs/dev/LicenseHeaders.md
diff options
context:
space:
mode:
authorTimothy Farkas <timothyfarkas@apache.org>2018-03-15 14:27:55 -0700
committerAman Sinha <asinha@maprtech.com>2018-05-11 14:56:07 -0700
commitdc8d0107cb6a4a31e1beaa9c98d6e51cf7486a67 (patch)
tree2457e58b5e5210c6d52c827c3ce6e8ad5ee8e48a /docs/dev/LicenseHeaders.md
parent8a1a7c53fb211dcba6e7b2f2ce90c28af4b9c518 (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.md34
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.