Code review comment for lp:~jameinel/bzr/2.0.5-switch-unicode-317778

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

It looks basically ok.

If there is an existing test we can amend to use a unicode name that would be nice.

It seems like really this should be lifted out of branch itself though.

review: Needs Fixing

« Back to merge proposal