Josh Matthews

Assertion errors should report a broken state to the relevant PR

4

I was left scratching my head when homu stopped making any progress while there was still an ongoing queue of approved PRs. It turns out in https://github.com/servo/servo/pull/21586 that I had...

It's common to use delegate+ in a PR, then have the PR author use r+ instead of r=delegator. We should just rewrite that behind the scenes.

Intermittent "Test was successful, but fast-forwarding failed: 422 Update is not a fast forward"

1

eg. https://github.com/servo/servo/pull/9911#issuecomment-204465843

Make homu testable

12

Right now homu is built to interact directly with the github API and buildbot. We want to refactor homu so that its hooks produce a list of actions, and those...

try command is ignored after failed merge

1

The correct command is `try clean retry` for some reason.

try builds should form an actual queue

1

When homu receives a request for a try build, it tries to fulfill it as soon as possible. This means that any builds that have not yet started for an...

The count labeled "approved" on http://servo-master.servo.org:54856/queue/servo is something like `(try success + pending + approved)` (although maybe with an off by one error?); when you just want to see how...

Make r+ by a delegatee infer the delegator

2

This will prevent a common source of incorrect historical metadata in the commit log.

Odd "approved by" message

1

See https://github.com/servo/servo/pull/10650#issuecomment-210875169: ``` :pushpin: Commit d09a497 has been approved by `` ```

This has always been a frustration for me, since the first platform that fails gets a comment with a link and you have to hunt for other ones if there...