[phobos] Automatic Pull Merging!
Brad Roberts
braddr at puremagic.com
Fri Nov 15 00:17:03 PST 2013
I've just put the automated pull merging code into production and visible to everyone. The rough
details:
1) You have to be logged in and using https (should be automatic) -- note the new login link in the
upper left corner. The website is now integrated with github's oauth system and will direct you
over there to accept this login. It requires write credentials because the merge operation is
performed as the person who requests the merge. This is subject to debate. It's possible to switch
to read-only credentials and have all merges done as me, with a merge comment noting who did the
commit -- but I prefer it NOT be me but rather be you guys.
2) To request that a pull be merged, head through link on the page with all the pull requests and
their status (https://d.puremagic.com/test-results/pulls.ghtml?projectid=1) to the test history link
(second column.. the one with the username/branch titled link). At the top of that page, for anyone
(logged in or not) is a new row labeled "Auto Merge" with a yes/no indicator. If you're logged in
you will see a Toggle link.
3) For any request which is already completely tested, it will merge right then and there (though no
visual indicator on that page until a few minutes later when the system has detected the completed
merge). For any incompletely tested pull request, once it passes all 10 platforms, it will be
merged in.
4) If the pull is updated to a new revision while waiting for a complete set of tests to pass, then
the auto-merge state is dropped and must be re-enabled.
Two known issues:
1) the post-login step is always back at the default page rather than where you were when you logged in.
2) anyone who is logged in can toggle it. Once the pull passes all tests, when it tries to perform
the pull, it notices the requester can't and yanks the flag, not performing any merging. Since
someone who could have rightly set it will be mislead that it had been set, that's pretty ugly.
There's every possibility that there's bugs in the system, so please tread carefully and let me know
if you see something acting fishy. Future work will add more integration with github to make it
easier to interact with.
Happy merging,
Brad
(yes, I realize those docs, such as they are, really belong within the website.. it's on the list)
More information about the phobos
mailing list