[ 
https://issues.apache.org/jira/browse/DRILL-6320?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16435826#comment-16435826
 ] 

ASF GitHub Bot commented on DRILL-6320:
---------------------------------------

Github user vrozov commented on the issue:

    https://github.com/apache/drill/pull/1207
  
    -1 on removing apache-rat maven plugin. It is an Apache plugin to invoke 
"Release Audit Tool". More plugins can be added to check license headers if 
they do a better job in addition to apache-rat.
    
    Both apache-rat and any other license verification plugins can be disabled 
by default (this will allow to run `mvn -T 2C clean install` if necessary) and 
only enabled in Travis CI.


> Don't Allow Javadoc comments for license headers
> ------------------------------------------------
>
>                 Key: DRILL-6320
>                 URL: https://issues.apache.org/jira/browse/DRILL-6320
>             Project: Apache Drill
>          Issue Type: Improvement
>            Reporter: Timothy Farkas
>            Assignee: Timothy Farkas
>            Priority: Major
>              Labels: ready-to-commit
>             Fix For: 1.14.0
>
>
> Currently some license headers are in a javadoc comment instead of a normal 
> comment. This is not good since we don't want to include spurious license 
> headers in java docs when we publish them. The fix would be to change the rat 
> plugin configuration to not allow java doc comments.
>  
> *For documentation*
> Min required maven version has changed.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to