I'm a software developer and we've recently implemented code reviews. The process should go like this:
- The developer submits a pull request.
- The developer's partner reviews the code, and approves it/leaves suggestions
- Once the partner has approved it, one of our two managers/admins approves it and it can then be merged into our main branch
The problem is, as we near our deadline, one of the managers constantly removes me from my partner's PR review lists, approves it, and merges the code in without reading it. My partner was hired as a senior developer (but has worked on this codebase for less time than me) but based on what I've read/questions asked writes infinite loops and is unaware how classes are instantiated. If I'm not able to get to the code review within an hour or two (which I don't find unreasonable), I will often then see that it has been approved/merged without my input. This obviously causes issues down the line.
How do I approach my manager about this situation without seeming like I'm attacking the other more senior developer?
**Non technical version: ** We have a process in place, which my direct manager often ignores because the other person hired was more senior and he doesn't check her work, even though it is lacking. How do I deal with this?