[prev in list] [next in list] [prev in thread] [next in thread] 

List:       mutt-dev
Subject:    Re: The future of mutt... - intermediate aggregation
From:       "jpacner () redhat ! com" <jpacner () redhat ! com>
Date:       2013-10-24 9:01:54
Message-ID: 5268E202.3020409 () redhat ! com
[Download RAW message or body]

Hi Fredrik,

> If you need an automated tool to enforce formatting rules, doesn't that
> apply that your code review process is broken and you risc to slip in
> serious bugs? Shouldn't formatting rules be part of the ordinary code
> review process?

It depends. IMHO it should be, but if the project is small as mutt, then
this check would spare time of reviewer (the patch would be declined
automatically, so the contributor will be notified himself). If it's
feasible to automate things, do it.

Kind regards

-- Jan Pacner
[prev in list] [next in list] [prev in thread] [next in thread]