Hi,

as the subject says, pull requests can stay for days without review, and
I really don't like merging my own PRs without someone having looked
over it.

Reviewing documentation doesn't require any specific knowledge even if
being a native English speaker might be an advantage :-) Just tell the
author if you understand what he wrote, and if the language is correct.

Reviewing code is more engaged but requires IMHO only some Python coding
knowledge and the ability to detect bad coding style, and challenge the
author to make sure nothing has been overlooked and everything is
properly commented. It doesn't require deep knowledge of the code but
can be a great way to gain this knowledge.

You only need to review the PR which are _not_ WIP and where the
pipeline job was successful (green tick).

Thanks, Eric

Reply via email to