I have another patch on top of D20998 which brings unit over, but it's currently so broken that it can't diff itself.

I'm also going to collapse as much of experimental into master as I can here, make the minimum required PHP version 5.5 (see T11968), and attempt to collapse wilds.

Probably require PHP 5.4 regardless.

A problem in moving forward here is that we ultimately do very little complex data access in Phabricator, and what complex data access we do perform can often be faked. We likely have more use cases in Arcanist and provisioning code: API calls are slower, workflows are more parallel/interactive, and we can't just fake it all with AJAX.

https://discourse.phabricator-community.org/t/activities-notifications-spam/3286/ is likely a case where curl was not installed on the CLI, but was installed in the web SAPI.

A saved state is likely something like this:

I'd like to leave behind a "Results were GC'd on X/Y/Z" message when we GC these so it's clear what's going on.

(See also T13160, and D20568 for additional rationale.)

See also PHI1630.

To attempt to justify micro-optimizing this loop:

See also PHI1628, which reports that a 4MB blob of test details is slow to render.

I might make some more tweaks to the UI, but this looks like it's in better shape now.

The actual replacement is Authorization: token <token>, I believe:

I just wanted to hold it across the release cut, it's headed to master shortly.

@epriestley what's blocking landing this diff?

Both of these hosts restarted cleanly.

AWS is also rebooting web007.

See https://discourse.phabricator-community.org/t/herald-rule-not-adding-auditors-to-commits-following-upgrade/3505 for one bug with this.

Sounds good. It felt slightly odd to me at first, too, but I think I also got used to it.

It wasn't a major UI issue to start with and yes, you get used to it. I would ignore.

This appears to be working. T13481 fixed one straggling issue with changes to remote handling in very old versions of Git but nothing else has cropped up.

The empty space means "this is a normal commit with no special audit state", and the column collapses if no commits have issues (see this task for an example). That seems fairly reasonable to me?