[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [E-devel] .cvsignore rampage.
Justin Patrin wrote:
> On 2/24/06, Kevin Brosius <firstname.lastname@example.org> 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.
> And you can always diff -b (--ignore-space-change).
Yeah, well, web webcvs supports that option, I'll buy that argument.
> I agree, though, formatting changes and real edits should not be in
> the same commit.
> Justin Patrin