Code review comment for lp:~vila/bzr/426344-check-pending-merges

Revision history for this message
John A Meinel (jameinel) wrote :

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Vincent Ladeuil wrote:
> Vincent Ladeuil has proposed merging lp:~vila/bzr/426344-check-pending-merges into lp:bzr.
>
> Requested reviews:
> bzr-core (bzr-core)
>
> This fixes the last two places where we were checking uncommitted changes
> while not taking pending merges into account: merge and remove-tree.
>

 review: approve
 merge: approve

However, this is looking more and more like we should have a simple:
  working_tree.has_changes()

Which takes no target, and grabs its own basis_tree, and looks at its
own pending merge list. Which I asked for in the beginning, though, so
perhaps I'm biased.

John
=:->
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (Cygwin)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iEYEARECAAYFAkqmmPcACgkQJdeBCYSNAANcQACfTLaWXv/YvIx79zte6lhB2CWg
3JUAoJKgX7MQd/GI06BE4XCc0vKn3qkJ
=LDTT
-----END PGP SIGNATURE-----

review: Approve

« Back to merge proposal