Code review comment for lp:~benji/lpsetup/add-get-subcommand

Revision history for this message
Gary Poster (gary) wrote :

I added a kanban card for adding he lightweight checkout support.

There are a number of other important details in the bug/kanban description that have not yet been implemented, as far as I can tell. The repository argument, which should mean that it is created only if needed, looks missing to me–it is always created. The destination directory is hard coded. In general, the command needs to be reviewed to fit into the requested plan. If the plan is unclear, that's very understandable, but we still need to try and share knowledge and follow it.

This is probably from the original, but i'd also like to look through the whole file to make sure bar terminology is used correctly, be i branch, repository, or checkout.

« Back to merge proposal