highfive icon indicating copy to clipboard operation
highfive copied to clipboard

r? doesn't work in a review comment

Open varkor opened this issue 6 years ago • 5 comments

When making a review, if r? is used in (at least the main comment of) the review, @rust-highfive won't pick up on it.

varkor avatar Nov 24 '18 21:11 varkor

There's at least two issues here:

  1. The webhook in each of the repositories using Highfive is not configured to send review events to Highfive.
  2. Even if Highfive were receiving those events, it would ignore them.

Fixing the first one takes changing settings in each of the repos using Highfive. The second one is some dev work in this repository.

Alternatively, I have been thinking about proposing changing Highfive to be a GitHub app, which would make situations like this easier to address, but there are other issues to handle before that is possible.

davidalber avatar Nov 25 '18 05:11 davidalber

For exposition, do you have a link to an issue where you used r? in a review comment?

davidalber avatar Nov 25 '18 05:11 davidalber

I originally posted it in https://github.com/rust-lang/rust/pull/56203#pullrequestreview-178062613 (but since then, I removed the r? in an edit once I saw that it didn't work).

varkor avatar Nov 25 '18 13:11 varkor

(Oh, I just realised this was about r?, not r+.)

dwijnand avatar Nov 25 '18 15:11 dwijnand

Just as a note, changing the configuration of the webhook across all repos is now trivial thanks to the update-webhooks.py script.

pietroalbini avatar Dec 16 '18 21:12 pietroalbini