Code review comment for lp:~gundlach/nova/controllers-in-api

Revision history for this message
Jay Pipes (jaypipes) wrote :

Yep, looks great.

Wondering why you chose to make _id_translator.py prefixed with an underscore...is that file meant to be a private utility? Is there some reason you didn't want it to be part of the set of "public" Python classes?

review: Approve

« Back to merge proposal