So, according to the fine manual (https://secure.phabricator.com/book/phabricator/article/differential_land/) the "Land Revision" button on the Phabricator user interface depends on setting up:

• Drydock repository automation : https://secure.phabricator.com/book/phabricator/article/drydock_repository_automation/
• Which depends on setting up Drydock working copies : https://secure.phabricator.com/book/phabricator/article/drydock_working_copies/
• Which depends on setting up Drydock hosts : https://secure.phabricator.com/book/phabricator/article/drydock_hosts/
• Which depends on Almanac : https://secure.phabricator.com/book/phabricator/article/almanac/

I did all these steps by hand way back when, then the worker I had that setup on was reinstalled and the setup broke (obviously).

All these Phabricator components have big scary red banners telling you not to use them and that they're broken and that they might change without warning. However, since I last touched them in 2016, they don't seem to have changed an inch, so they're probably more """""stable""""" (i.e. dead) than upstream claims.

I feel kinda awful setting up such a Rube-Goldberg machine just to be able to click a button to merge a f...abulous patch on a f...riendly web interface. I'd rather invest some time on letting a comment trigger a jenkins job that would push the patch once the tests pass.

(looks like Herald can't easily trigger on a comment, because that'd be too easy)

Thanks for the investigation @olasd.

I think we need uniformity more than anything else. So if, for any reason, the web-based landing isn't an option, we should remove it from where it exists (I think that's just the Web UI repo, but I haven't checked.)

I'm not a fan of automated merging when the test suite passes. I think merging code from third parties should always be a manual step by someone. If it can be made a Web UI button, even better, if not we will just standardize to the local arc workflow, which I've recently documented in the wiki.

And of course it's not OK if we need to configure the thing manually for all new repos we create in the future, that would be too much of a burden (hence we'll surely forget about it).

In T1689#31539, @zack wrote:

I'm not a fan of automated merging when the test suite passes. I think merging code from third parties should always be a manual step by someone. If it can be made a Web UI button, even better, if not we will just standardize to the local arc workflow, which I've recently documented in the wiki.

That's not what I suggested. My suggestion was having *someone* trigger the merge with a comment on the diff, once the tests pass.

And of course it's not OK if we need to configure the thing manually for all new repos we create in the future, that would be too much of a burden (hence we'll surely forget about it).

Yeah, well, with Phabricator I don't think we'll avoid that.

In T1689#31540, @olasd wrote:

IMy suggestion was having *someone* trigger the merge with a comment on the diff, once the tests pass.

Oh. Then yes, that would be cool !