<div dir="ltr">Hi guys,<div>Sorry I was not able to attend.<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br>    Bug 2340 Update (djmitche, 16:37:19)<br>    Turning on protected branches in Github (djmitche, 16:39:24)<br>        <a href="https://github.com/blog/2051-protected-branches-and-required-status-checks" target="_blank">https://github.com/blog/2051-protected-branches-and-required-status-checks</a> (djmitche, 16:39:46)<br>        proposal is to protect the 'master' and 'eight' branches, requiring that travis / appveyor / circle all succeed for a PR to be merged (djmitche, 16:43:10)<br>        AGREED: everyone present seems to agree this is a good idea (djmitche, 16:43:20)<br></div></blockquote><div><br></div><div>I don't disagree with that. I have been implementing that rule manually already. Please let me know if I hadn't.</div><div><br></div><div>We had a travis regression last week, which was to my understanding due to a conflict between two PR. The code was mergeable, but the test introduced in one was not compatible with the code change introduced in the other one.</div><div><br></div><div>We have a lot of activities those last few weeks. I hope we will continue on that trend with the forthcoming nine release.</div><div><br></div><div>I try to merge the code as soon as possible with the rules (of thumb) as follow:</div><div>- If the change is trivial enough and looks good to me, I just wait for travis and hit the merge button</div><div>- If there is more debatable changes, I :+1: and wait for a few days so that other ppl have the time to review it.</div><div><br></div><div>Please don't hesitate to help applying this process as well. Hit the review and merge buttons when you think it is appropriate!</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br>    mainstreaming buildbot_travis (djmitche, 16:44:50)<br>        proposal is to move <a href="https://github.com/tardyp/buildbot_travis" target="_blank">https://github.com/tardyp/buildbot_travis</a> to the buildbot org, and make it a buildbot product (djmitche, 16:49:30)<br>        AGREED: will move buildbot_travis into the buildbot org (djmitche, 16:50:17)<br>        ACTION: tardyp to move buildbot_travis (djmitche, 16:50:33)<br></div></blockquote><div><br></div><div>Done. </div><div><br></div><div>jc2k has been very helpful at transferring the ownership of the original fork of buildbot_travis to the buildbot organisation. With this we can keep our SEO rank, fork list, and starlist, as github is setting up redirects as appropriate.</div><div><a href="https://github.com/buildbot/buildbot_travis">https://github.com/buildbot/buildbot_travis</a></div><div><br></div></div></div></div>