We review code to put another set of eyes on code that is going to production. More people that look at code means that you get different views on how to do things. It also serves as a form of cross-training and collaboration that allows us to share techniques.
You can see an outstanding list of code reviews.
You can post to Microsoft Teams to request code reviews. However, feel free to call out individual people if you want someone specific to look at the code. When modifying code that someone else usually maintains, you should specifically request a code review from that person and, if possible, wait until that person has reviewed the code before continuing with the merge request.
If someone has added a comment, it should be marked as resolved before continuing with the merge request. If it cannot be handled immediately, you should mark it as an issue before continuing with the merge.
You can see the full list of open issues in IT Partners.
There will be cases when you cannot review code (either everyone else is out or busy), or times when code reviews are impractical. If you need to skip code review, please post in the ITP Developers - Code Review and Conversation channel.
All pulls are sent to the ITP Automated Channels and Github team, under the Github Pull Requests.