opendistro-for-elasticsearch / index-management

🗃 Open Distro Index Management
https://opendistro.github.io/
Apache License 2.0
115 stars 45 forks source link

[BUG] PR with no license headers didn't fail checks #144

Open dbbaughe opened 4 years ago

dbbaughe commented 4 years ago

Describe the bug This PR had some files with missing license headers. It should fail an automatic check instead of relying on human verification.

Expected behavior Run ./gradlew build with a source file w/ no License Header and fail.

dbbaughe commented 4 years ago

Skips when there are no java files which is why task isn't running https://github.com/elastic/elasticsearch/blob/master/buildSrc/src/main/groovy/org/elasticsearch/gradle/precommit/LicenseHeadersTask.groovy#L71