highfive

highfive copied to clipboard

As discussed in this week's Servo meeting, it would be great if highfive added a comment when new dependencies are added. To go above and beyond on a language-specific basis,...

edunham

jdm

Add S-fails-tidy when appropriate commit status changes

5

`"context": "homu"` and `"state": "failure"` appear to be key.

jdm

List submodules being updated by a PR

1

Every time a PR updates and includes a submodule change, we should add a comment to draw attention to that fact.

jdm

Look for `unsafe {` and `unsafe fn` instead of just `unsafe`.

jdm

It doesn't have to be good or bad, just a quick glance of "oh, this PR sure add a lot of new clone calls." Look at the number of `.clone()`...

jdm