Code review comment for lp:~craighewetson-deactivatedaccount/bzr/update_with_local_commit

Revision history for this message
Craig Hewetson (craighewetson-deactivatedaccount) wrote :

I've tested this on my ubuntu machine and it works very well. I fails (with a clear error message) to update if the repository has local commits and uncommitted changes. It will inform the user to locally commit them before doing the update.

I suppose there could be a future improvement, and that is to check if the uncommitted changes where files that where locally committed. But I reckon that we might miss certain conditions (renaming etc) and this might cause it not to be full proof.

I suppose in the future Bazaar can handle this in a better way and try to merge the files for the user, but until then lets not allow the user to hang himself.

Please review and let me know if its good enough to go into trunk

« Back to merge proposal