Hi Leif, There's a .rat-excludes in the PR I posted today (https://github.com/apache/incubator-trafficcontrol/pull/121) -- I'm sure it's too liberal, but wanted to make sure we don't go backward..
-dan On Mon, Dec 5, 2016 at 5:08 PM, Leif Hedstrom <[email protected]> wrote: > >> On Dec 5, 2016, at 3:48 PM, Dan Kirkwood <[email protected]> wrote: >> >> Hi all.. We haven't really established a process for this, but to >> keep in compliance with Apache license guidelines, each source file >> should have the Apache license comment -- normally at the head of the >> file, but I think that's somewhat flexible. >> >> Still going thru files adding them, but when any new files get added, >> they really should have that header in them already. >> >> What do you all think of establishing a guideline that any PR is not >> merged until the license is present in each source file added? > > > Very much +1. There are exceptions, at which point you would add them to the > RAT exclude file, but use that as sparingly as possible :). > > — Leif > >> >> -dan >
