Hi,
We have an agreed Eclipse template, why can't people have the code
formatting done using the code style in the 1st place?
I agree not to reformat the whole file at the same time when we fix the
code, but I don't think we go too far. I have a few things set up
automatically in Eclipse:
* Organize the imports
* Convert the tab/space
It helps me a lot to maintain the clean code with disturbing the diffs too
much.
The other thing, can we reformat the whole code base before we cut a RC so
that we can have good style moving forward?
Thanks,
Raymond
--------------------------------------------------
From: "ant elder" <[email protected]>
Sent: Wednesday, June 24, 2009 7:02 AM
To: <[email protected]>
Subject: Re: Commit comment quality?
On Wed, Jun 24, 2009 at 2:27 PM, Luciano Resende<[email protected]>
wrote:
On Tue, Jun 23, 2009 at 11:36 PM, ant elder<[email protected]> wrote:
Does anyone disagree that commits should not mix formatting and code
changes?
There are some settings on the Eclipse templates we use that do some
conversion automatically (e.g tabs for spaces)... are those accepted
?
I'd don't think those should be done automatically and from other
projects its standard practice to to keep those separate in another
commit. We're all busy so reviewing changes should be able to be done
as quick and easily as possible, so ideally just read the comment and
a quick flick over the diff, if that diff includes many other lines of
change even just for tab/space changes it makes it harder than
necessary to see the actual code lines changed.
...ant
