Code review comment for lp://staging/~martin-hilton/juju-deployer/speed-up-inheritance

Revision history for this message
Roger Peppe (rogpeppe) wrote :

LGTM except that it really should have a test (ideally one that goes
into an infinite loop before this fix).

Also, given that it's almost certainly an infinite loop (correctness)
problem, the change description seems inaccurate.

« Back to merge proposal