Merge Request Code Review
Review a ~70-line merge request of varying complexity and decide whether you would approve it. Identify both obvious and subtle issues that should be addressed before merging, and explain what questions or feedback you would give to the author.
Asked at:
JPMorgan Chase
Question Timeline
See when this question was last asked and where, including any notes left by other candidates.
All Regions
Late October, 2025
JPMorgan Chase
Senior
Your peer is trying a merge request. What would you ask them? Would you approve it? etc. Code is about 70 or so lines. Some of it is complex. There are obvious things to call out and some are not.
Hello Interview Premium
Your account is free and you can post anonymously if you choose.