I read this where it says “To make sure the right people see your patch, please select suitable reviewers …” but I feel like this is applicable to Phabricator, but not to Github, since I don’t see any way of assigning reviewers myself. Maybe I’m missing something, but it’s not clear to me how should I request for reviewers.
Thanks for the clarification (and for the reviewers ).
It makes sense, but it didn’t feel good doing it if that was not documented anywhere. Now at least it is documented here, although it might be worth updating the docs so that new people can see it. Thanks!
Yes, please do update the docs! We’ve seen this question a few times, here and on Discord. If you don’t have the time to update the docs, maybe it’s worthwhile to quickly create a github issue for it, marking it as a good beginner issue (by adding the “good first issue” label.
@pablo If you have the time, could you take a look and see if that would have helped you? (on the assumption that our github guide gets updated to answer the reviewers question)
Your perspective on the wording would be very valuable.
@DavidSpickett I had a look and it looks good to me. The wording it’s clear and the information is helpful for people that is not familiar for the process.
).