[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [E-devel] .cvsignore rampage.



On Fri, 24 Feb 2006 13:57:18 -0800 "Justin Patrin"
<papercrane@gmail.com> wrote:

> On 2/24/06, Kevin Brosius <cobra@compuserve.com> wrote:
> > David Seikel wrote:
> > >
> > >
> > > I will probably go on a cvs rampage this weekend.  I'll start with
> > > getting the .cvsignore files in order, as they are in need of
> > > some tlc, and I might end up with getting engage and enotes up to
> > > date with the recent EWL API change if their authors don't beat
> > > me to it.  Somewhere in the middle I am likely to apply indent to
> > > whatever I'm considered to be responsible for.
> >
> > Don't commit indent runs and code changes in the same cvs
> > transaction. It makes it impossible to tell what changed.
> 
> I agree, though, formatting changes and real edits should not be in
> the same commit.

There is no overlap between the formatting and code changes I
mentioned.  I'm in total agreement with you anyway.

Attachment: pgpQPep5enXIY.pgp
Description: PGP signature