EOD status 19-06-2012
William Reade
william.reade at canonical.com
Tue Jun 19 17:27:49 UTC 2012
FWIW:
https://codereview.appspot.com/6297101 (usable-deploy-command) is marked
WIP, but is IMO ready for *review*; just not for merging (there are
failing tests for cs: URLs, and I'm not sure how to exercise the code
without hitting the real CS. Suggestions appreciated.) Maybe it
shouldn't be WIP? But it contains a couple of `c.Fatalf("not tested")`s,
so I don't feel it deserves not to be. Gaah.
https://codereview.appspot.com/6305107/ (unit-scope) is still out for
review; I think/hope it's a small and sensible chunk of functionality
that won't be too controversial, but it's been out for a day, so I worry
that there's some problem with it that people aren't telling me
about ;).
https://codereview.appspot.com/6312048/ (relation-type) hopefully
incorporates the feedback received via IRC adequately and has also been
waiting (a bit less than) a day.
The first CL is less urgent than the other two; but I loathe the
experience of building pipelines on top of pairs of prereqs, so I'm
kinda blocked on one of them (either one; doesn't matter which; I can
deal with simple pipelines just fine).
Happy evenings (mornings?) all.
Cheers
William
More information about the Juju-dev
mailing list