Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update license headers and add validation to 'check' #170

Merged
merged 2 commits into from
Sep 5, 2016
Merged

Update license headers and add validation to 'check' #170

merged 2 commits into from
Sep 5, 2016

Conversation

tolbertam
Copy link
Contributor

@tolbertam tolbertam commented Aug 9, 2016

Ahead of the incoming PR for #76 , I thought it would be good to add license-gradle-plugin for adding, updating and validating license header presence in all source files.

License validation will be done automatically as part of the check task. The build will fail if the header is missing from any file.

To update all files with the LICENSE header, simply run:

./gradlew licenseFormat

See b4d046a for the gradle changes specifically.

@tolbertam
Copy link
Contributor Author

tolbertam commented Aug 9, 2016

downgraded to 0.11.0 (from 0.13.1) of the plugin since the latest version requires JDK 7 (hierynomus/license-gradle-plugin#35). At some point we could consider removing JDK6 build support and use something like animal-sniffer to ensure JDK6 support is retained..can look into that if you think that is a good route.

@chbatey
Copy link
Member

chbatey commented Sep 5, 2016

Thanks @tolbertam

@chbatey chbatey merged commit bf46da6 into scassandra:master Sep 5, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants