I just removed a binary file (a .jpg file) and the command arc patch failed as following :

You need or we need? Phabricator should only require one individual from the Blocking Reviewer project to satisfy the accept condition.

OK Chad, but if I add a specific blocking reviewer, another reviewer will be able to accept the revision but this one will stay in status "need review".
And if I add all the members of the code review committee as blocking reviewer, I gonna need all the blocking reviewer to accept the revision.
I'm sorry if my original message was not enough clear.

Yes I'm already subscribed to T4887 but I don't want to add a "blocking reviewer" as any member of the code review committee can accept the revision. And the task T731 is great but is complementary to what I'm asking. The case I describe is not covered by T731.

It's a real problem when for example we store .mo files (binary translation files) or images, pdf, ...

In waiting for an official fix from Phabricator teams, if someone knows how to quickly fix this issue, it should be great to share it.

I confirm having the same problem with phpcs linter. Each time phpcs raises warnings or errors, arcanist throws that exception and in my code review the Arcanist Lint Result remains into "Waiting for Message" status. You should fix it quickly as phpcs is not usable anymore :(

ok thanks so this feature is not for me ;)
you can close the ticket

So when is it expected ?

I actually followed this tutorial (very good tutorial) and it worked perfectly.
But the problem has never been to make work Jenkins and phabricator together but to avoid developers to land their branch while Jenkins is run

We use to run jenkins build (in waiting for harbormaster :)) after each diff creation (we implemented the event TYPE_DIFF_WASCREATED).
When jenkins finishes its build it will add a comment on the revision or reject it if a problem occured.

it should be nice to create an event listener "before landing"

✘ Merged into T4555.

◀ Merged tasks: T4348.