Since markdown is intended to be rendered when read and is rendered on GitHub I think it's also important to include the short license text not in a comment.
On a related note I think it's frowned upon to have as many wildcarded rat check excludes as are currently there in the pom. It's a hassle but it's safer to do the excludes per file in the sub-modules. -joey > On Dec 21, 2016, at 4:18 PM, Otto Fowler <[email protected]> wrote: > > Looks like they use XML escapes <!— —> > > > > On December 21, 2016 at 15:48:07, Casey Stella ([email protected]) wrote: > > We should do this. I was not aware that Markdown took comments. Every > file which is capable of handling comments should have an apache license, > IIRC. > > On Wed, Dec 21, 2016 at 3:45 PM, Otto Fowler <[email protected]> > wrote: > >> I was just looking at the NiFi source, and noticed that they include the >> license header in their readme files. Is this something that we are >> supposed to be doing or is it optional? >>
