[action-required-review] request review fails when required reviewer is PR author about jetpack

[anomiex]

I don't think it would be worth adding logic for an actual "fallback" (i.e. "X, or Y only if X is the author") versus just configuring the action with "any-of: X, Y". That would also handle situations like when X is on vacation.

The fix looks ok to me, although I might instead do the check against author at

jetpack/projects/github-actions/required-review/src/request-review.js

Line 20 in e9150a3

userReviews.push( t.slice( 1 ) );

while processing the array instead of pre-filtering the array.

If you'd make a PR against the file linked above in this repo, we can get it reviewed and merged.

[evan-gray]

Successfully tested a fix.

[jeherve]

In your fix, you're skipping the required review when the required reviewer is the PR author. On one hand that makes sense. On the other, I wonder if it wouldn't be best to avoid those scenarios altogether by having a fallback reviewer (team or person)?

[evan-gray]

@anomiex I had considered that but then the message Requested review(s) from... would include the name of the skipped author. Also concur that fallback does not feel necessary as there, ideally, should be redundancy in a reviewer list. Though it is interesting that CODEOWNERS just allows any reviewers if the sole owner is also the PR author 🤷

Will open a PR, just wasn't sure if other changes were necessary given the mirror repo setup.

[anomiex]

I had considered that but then the message Requested review(s) from... would include the name of the skipped author.

Good point, I missed that.

[evan-gray]

Awesome, thanks for the assist @anomiex 🤝