Mitch Shanklin created BEAM-860:
-----------------------------------

             Summary: Move Apache RAT license check out of release profile
                 Key: BEAM-860
                 URL: https://issues.apache.org/jira/browse/BEAM-860
             Project: Beam
          Issue Type: Improvement
            Reporter: Mitch Shanklin
            Assignee: Mitch Shanklin
            Priority: Minor


Currently Apache RAT only checks licenses as a part of the release profile. 
Since the contributor's guide advises users to run 'mvn clean verify' without 
the release profile locally, this means that missing licenses on files are 
caught by Jenkins when they could be caught locally, saving a cycle.

Since RAT runs quickly, there doesn't seem to be a great reason to keep that in 
the release profile. Seems similar to checkstyle in many respects, which is not 
part of the release profile.

See https://github.com/apache/incubator-beam/pull/1199#issuecomment-256802048 
for discussion of this issue.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to