Code review comment for lp:~nii/nova/nova-deployment-tool

Revision history for this message
Soren Hansen (soren) wrote :

By accepting this into trunk, it's implied that it will be kept up-to-date, given that people checking out trunk will actually be able to find it as part of the checkout. I know that *I* will not be maintaining it. Unless you can find a nova-core person that will volunteer to keep this up-to-date (or at the very least pull it out once it becomes out-dated (which won't be long at the pace this project moves)), I can't approve it. Sorry.

Just to be clear, this has nothing to do with the quality of your work (I've only glanced at it). This is about ensuring the future coherence of trunk.

Disapproving for now, but if you can find a nova-core person that will promise to keep it up-to-date, I'm very willing to reconsider.

review: Disapprove

« Back to merge proposal