Code review comment for lp:~jameinel/bzr/2.1.0rc1-set-mtime

Revision history for this message
Vincent Ladeuil (vila) wrote :

Sounds good !

Just a nit:

8 +* Operations which update the working tree should now create all files
9 + with the same mtime.

When reading this I had slight doubt about whether modified files were also concerned.

I know they are, but saying create/modify instead of just create may make it clearer.

review: Approve

« Back to merge proposal