[
Date Prev][
Date Next][
Thread Prev][
Thread Next][
Date Index][
Thread Index]
[
List Home]
Re: [egit-dev] Granularity of changes
|
torsdagen den 5 augusti 2010 16.37.11 skrev Benjamin Muskalla:
> Hey there,
>
> I was working on a bug yesterday (bug 307112 for the sake of
> completeness). The problem:
> Not only bug fixing was neccessary but also several refactorings that
> make the code more readable. First off, I extracted several methods,
> commit, then refactored some of the logic how the dialog behaves,
> commit, and finally fixed the bug itself.
> In the past, I squashed all commits together and pushed them upstream.
> But I think the right thing to do is to push all of the steps to the
> repo to a) have a history that makes it easy to follow the development
> and b) give others a better possibility to follow my changes.
> Pushing all commits means that we need to review step by step instead of
> one bug chunk.
>
> What's your opinion on that? I'd prefer pushing the steps as separate
> commits but if the rest of the team thinks that the review overhead is
> too much, that's ok with me.
Small commits speeds facilitates review both regarding speed and accuracy. Big
patches tend to spend a long time in Gerrit before being accepted.
-- robin