From: William Duclot <william.duclot@ensimag.grenoble-inp.fr>
Date: Thu, 19 May 2016 18:10:29 GMT
Subject: Re: [PATCH/RFC] Add userdiff built-in pattern for CSS code
Message-ID: <666636251.5761714.1463681429687.JavaMail.zimbra@ensimag.grenoble-inp.fr>
In-Reply-To: <CAPig+cQcK4x+u=PjxGAAPnAgqCAxLHrvbH8Pqgjgd9aAe85g7g@mail.gmail.com>

> On Thu, May 19, 2016 at 10:45 AM, Matthieu Moy
> <matthieu.moy@grenoble-inp.fr> wrote:
> >> Add the info in documentation that CSS is now built-in.
> >
> > This doesn't add much to the patch (we can already see that from the patch
> > itself). I'd remove it.
> 
> I think you meant to say this "doesn't add much to the *commit
> message*", and that the sentence should be removed from the commit
> message, while retaining the actual documentation update in the patch.
> 
That's of course how I understood it, but thanks for pointing it out !

Beside that, all of Matthieu Moy comments seem correct to me and will be taken into account.