Code review comment for lp:~jelmer/bzr/tweak-revert-help

Revision history for this message
Martin Pool (mbp) wrote :

Thanks for that.

As discussed on the list, defining the command as "revert files ..." is a bit circular, so perhaps it's better to say something like "set files in the working tree" or something.

 To remove only some changes, without reverting to a prior version, use merge instead.

Perhaps that should be made more clear that revert will normally discard any uncommitted changes.

Rather than 'last committed revision' we should probably say 'working tree basis revision', which is more accurate, and we should probably mention this never changes the basis revision.

Perhaps we should contrast revert to 'update', which carries uncommitted changes, which goes to the branch tip revision by default, and which does change the basis revision.

« Back to merge proposal