You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@beam.apache.org by "Mitch Shanklin (JIRA)" <ji...@apache.org> on 2016/10/28 18:54:58 UTC

[jira] [Created] (BEAM-860) Move Apache RAT license check out of release profile

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)