GGitLab·DSAEngineerOnsite – Coding (Practical)
Review a Merge Request on a Small App
Problem
You receive a merge request (sent 72h prior) against a small app. Review it as you would any contribution: correctness, tests, readability, and suggest improvements.
Example
Identify a missing edge case and a test gap; propose changes
Constraints
- Tailored to the role's language
What GitLab looks for
GitLab's signature MR-review round. Ask 'How did you identify the issue? Would another approach be better? Why this method?' — answer with reasoning.
added 6 days ago