Code review comment for lp://staging/~gary/juju-core/more-info

Revision history for this message
Gary Poster (gary) wrote :

Thank you both very much for the prompt review.

On 2013/04/01 22:52:58, fwereade wrote:
> LGTM -- maybe ping me tomorrow about status and status info?

OK, will do.

Short answer is that, if we were not under time pressure, I would have
zero concerns. With the time pressure, and the fact that I realized
last Friday that we are also already going to need to add separate
watchers for config and constraint docs, I'm a lot more nervous about
the change. If we could postpone the change to post 13.04 that would be
fabulous. If we can't, we'll just need to strategize, and see how
quickly Roger and we can whip up the necessary code on the GUI/API side.

https://codereview.appspot.com/8086045/

« Back to merge proposal