D2048: utils: copy util.py as utils/dateutil.py
durin42 (Augie Fackler)
phabricator at mercurial-scm.org
Thu Feb 8 09:15:14 EST 2018
durin42 added a comment.
In https://phab.mercurial-scm.org/D2048#34978, @lothiraldan wrote:
> In https://phab.mercurial-scm.org/D2048#34743, @durin42 wrote:
>
> > Can we do all of https://phab.mercurial-scm.org/D2048::https://phab.mercurial-scm.org/D2056 as a single change? What you've done means we have no useful blame information on everything done in https://phab.mercurial-scm.org/D2049::D0256...
>
>
> Moving function one by one proved very useful in the past when refactoring like this triggered subtle bugs in core or extension. Having each move on its changeset make it easier to bisect the part that breaks other and save a great amount of time when that happens.
Life's tough. I care more about the blame history on this code.
(Other reviewers are welcome to overrule me, but I don't find this argument even slightly convincing.)
REPOSITORY
rHG Mercurial
REVISION DETAIL
https://phab.mercurial-scm.org/D2048
To: lothiraldan, #hg-reviewers, durin42
Cc: durin42, mercurial-devel
More information about the Mercurial-devel
mailing list