Re: [DISCUSS] Removing Markdown files from rat exclusion

Bravo. On 1/2/18, 8:43 AM, "Justin Leet" wrote: The PR is merged into master, and all relevant PRs have a comment noting that adding the header is required. As a reminder, this means Apache headers are required on all markdown files and this will be

Re: [DISCUSS] Removing Markdown files from rat exclusion

The PR is merged into master, and all relevant PRs have a comment noting that adding the header is required. As a reminder, this means Apache headers are required on all markdown files and this will be enforced by rat. On Sat, Dec 30, 2017 at 8:33 AM, Justin Leet wrote:

Re: [DISCUSS] Removing Markdown files from rat exclusion

I've updated the PR to add the header to a new MD file that went in. I've also commented on all PRs that I saw that would potentially be problematic were they to go into master if they weren't merged first. Once the updated PR gets the +1's reaffirmed, it will be merged into master and Markdown

Re: [DISCUSS] Removing Markdown files from rat exclusion

I'm gonna let this percolate until Wednesday or so, assuming conversation doesn't reach a natural tipping point. I'm inclined to agree with Nick, but I also don't want to resolve anything in a way that even potentially causes master problems until at least after Christmas has a chance to settle