<div dir="ltr"><br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
- [ ] To get raw diff via git:<br>
<br>
git clone <a href="https://github.com/buildbot/buildbot.git" rel="noreferrer" target="_blank">https://github.com/buildbot/buildbot.git</a><br>
cd buildbot<br>
git fetch --all<br>
git diff origin/master...origin/bug2340<br></blockquote><div><br></div><div>I'll go for that one. This reminds me the good old time when I reviewed the nine branch from dustin, trying to understand what all this data api thing was about.</div><div><div><div><br></div><div><br></div><div>git diff buildbot/master master/docs/manual/worker-transition.rst</div></div><div><br></div><div>+  ``workdir`` property source from ``"slave (deprecated)"`` to</div><div>+  ``"worker (deprecated)"``.</div></div><div>While we are at it, I would remove those deprecated properties. They have been for a good while, and deserve a fair death.</div><div><div>+API changes between 0.9.0b7 and 0.9.0b8 (done without providing fallback).</div><div>+</div><div>+.. todo::</div><div>+</div><div>+   This whole section may be removed since it's not important for users</div><div>+   upgrading to 0.9.0.</div><div>+</div></div><div><br></div><div>Well, there as been already some people adopting nine before it was released (at least people in my extended team), so I think it should be good to keep it at least for a little time.</div><div><br></div><div><div>+     - ``/masters/n:masterid/workers/n:workersid``</div></div><div><br></div><div>its  workerid. I can verify that the code is okay, this is just a typo in the doc.<br></div><div> </div><div><div> git diff buildbot/master master/buildbot/test/unit/test_config.py</div></div><div><br></div><div><div>+from buildbot.test.util.warnings import assertNotProducesWarnings</div><div>+from buildbot.test.util.warnings import assertProducesWarning</div></div><div>[...]</div><div><div>+        with assertProducesWarning(</div></div><div>[nit] I think it is more unittest idiomatic to say 'with self.assertProducesWarning', means you would rather import something like:</div><div><div>from buildbot.test.util.warnings import warningAssertionMixins</div></div><div><br></div><div><br></div><div><div>diff --git a/master/buildbot/db/buildslave.py b/master/buildbot/db/buildslave.py</div><div>new file mode 100644</div><div>index 0000000..4f486b1</div><div>--- /dev/null</div><div>+++ b/master/buildbot/db/buildslave.py</div><div>@@ -0,0 +1,30 @@</div><div>[...]</div><div>+deprecatedWorkerModuleAttribute(locals(), _WorkersConnectorComponent,<br></div><div>+                                compat_name="BuildslavesConnectorComponent",</div><div>+                                new_name="WorkersConnectorComponent")</div><div>diff --git a/master/buildbot/db/buildslaves.py b/master/buildbot/db/buildslaves.py</div><div>deleted file mode 100644</div><div>index 8613f03..0000000</div><div>--- a/master/buildbot/db/buildslaves.py</div><div>+++ /dev/null</div></div><div><br></div><div>looks like db/buildslaves.py was renamed in the process of deprecation!</div><div><br></div><div><br></div><div>I think this whole thing is a great job! I join Dustin on the amazin' words.</div><div><br></div><div>As we know this beast is working (<a href="http://nine.bb.net">nine.bb.net</a>), IMHO we should merge it and treat the remaining todos in subsequent PR, which will be much much easier to review.</div><div><br></div><div>Cheers</div><div>Pierre</div></div></div>