[PATCH 1/3] settings: Move DEFAULT_FROM_EMAIL to the core settings section

Damien Lespiau damien.lespiau at intel.com
Tue Sep 29 00:35:53 AEST 2015


On Mon, Sep 28, 2015 at 02:55:27PM +0100, Finucane, Stephen wrote:
> > On Sun, Sep 27, 2015 at 11:07:55PM +0100, Finucane, Stephen wrote:
> > > > DEFAULT_FROM_EMAIL is actually a django setting, not a patchwork one.
> > > >
> > > > Signed-off-by: Damien Lespiau <damien.lespiau at intel.com>
> > >
> > > Since this is a purely syntactical change, I'm going to be pedantic
> > > and say 'email' should be title-case.
> > 
> > Sigh. These kind of things really aren't very productive.
> 
> Sorry, I know it's time consuming but someone has to do this stuff :(

That's where I disagree. I don't think anyone has to nitpick on
capitalization of a one word comment.

-- 
Damien


More information about the Patchwork mailing list