Code review comment for lp://staging/~vila/bzr/308472-info-version-smart-server

Revision history for this message
Robert Collins (lifeless) wrote :

I think this is ok; it would be better to have tests.

What would be better [and definitely needs tests] is that doing 'bzr info bzr://SERVER/' (that is, pointing at somewhere that isn't a branch or repo at all, but has a functioning server) would still do something useful.

-Rob

review: Approve

« Back to merge proposal