Code review comment for lp:~sinzui/charmworld/stale-metadata

Revision history for this message
Brad Crittenden (bac) wrote :

This change looks good.

I was initially concerned that the copy in 'process_charm' was not a deep copy given that the charm is a dict of dicts. But looking at the code, no lower-level dicts are mutated so it should be fine.

review: Approve (code)

« Back to merge proposal