This is an archive of the discontinued Mercurial Phabricator instance.

update: clarify calculateupdate() call sites by specifying argument names
ClosedPublic

Authored by martinvonz on Sep 25 2019, 5:38 PM.

Details

Summary

merge.calculateupdate() takes a lot of parameters and I get confused
all the time which is which.

See also b14fdf1fb615 (update: clarify update() call sites by
specifying argument names, 2017-02-09).

Diff Detail

Repository
rHG Mercurial
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

martinvonz created this revision.Sep 25 2019, 5:38 PM
pulkit accepted this revision.Sep 26 2019, 7:12 AM
This revision is now accepted and ready to land.Sep 26 2019, 7:12 AM