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

Re: [E-devel] .cvsignore rampage.



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.
> >
> > You have plenty of time to object.  B-)
>
> Just please... please... please...
>
> Don't commit indent runs and code changes in the same cvs transaction.
> It makes it impossible to tell what changed.
>

s/impossible/hard/

And you can always diff -b (--ignore-space-change).


I agree, though, formatting changes and real edits should not be in
the same commit.

--
Justin Patrin