[BEAM-860] Move Apache RAT license check out of release profile

• Type: Improvement
• Status: Resolved
• Priority: P3
• Resolution: Not A Problem
• Fix Version/s: Not applicable
• Component/s: project-management
• Assignee: Mitch Shanklin
• Reporter: Mitch Shanklin
• Created: 28/Oct/16 18:54
• Updated: 16/May/20 13:22
• Resolved: 01/Dec/16 21:53

Description

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.

Issue Links

• links to: GitHub Pull Request #1221