Sort Review Comments by Merge Impact
Categorize PR review comments so merge-blocking work is handled before optional improvements.
Sort each PR review comment by its merge impact. Blocking done gap Clarification needed Non-blocking improvement Separate follow-up This endpoint skips the role check named in the acceptance criteria. The bug fix has no regression test for the failing case. Why did you use a map here instead of preserving array order? Can you confirm whether this value comes from tenant config or environment config? The helper name could be clearer as buildInvoiceRows. Please add one sentence to the README because this flag is user-facing. This could eventually use the shared form layout component. We should split this whole module…
Sign up free — one personalized lesson every day, matched to your role and goals.
Already have an account? Sign in