Re: [PATCH]: Extend "Obvious fix" rule in HACKING
On Fri, May 30, 2008 at 12:33:43PM -0400, C. Michael Pilato wrote:
> Justin Erenkrantz wrote:
> > On Fri, May 30, 2008 at 5:01 AM, Stefan Sperling <stsp_at_elego.de> wrote:
> >> +<p>Whenever you are invoking the "obvious fix" rule, please say so in
> >> +the log message of your commit. For example:</p>
> >> +
> >> +<pre>
> >> + www/hacking.html: Obvious fix: Kill some typos.
> >> +</pre>
> > It's a bikeshed, but I'd do:
> > ---
> > Obvious fix.
> > * www/hacking.html: Kill some typos.
> > ---
> > I'm not a fan of having the rule conflated with the change
> > description. -- justin
> Bikeshed indeed, but sound reasoning nonetheless. +1.
Committed in r31538, with all suggested fixes.
Received on 2008-05-30 21:23:04 CEST
- application/pgp-signature attachment: stored
This is an archived mail posted to the Subversion Dev