Code review comment for lp:~fwierzbicki/txaws/break-out-parse-instance

Revision history for this message
Jamu Kakar (jkakar) wrote :

I guess the perfect fix would be to completely decouple the parsing
logic from the client, but that's probably more work than you want to
do. Anyway, if you're find with calling a private method I'm fine
with this branch, assuming it _parse_instance gets a docstring.

Nice one, +1! :)

review: Approve

« Back to merge proposal