The review
Four comments, not forty
What lands on the pull request. Every line points at a span and states what is wrong with it.
second-pass · PR #4182fail
reviewed 4 files, 34 lines
LOW src/analytics/track.ts:1
Edited a file outside the request
not among the 3 files the task named
MED src/billing/format.ts:6
Re-implemented an existing helper
renderAmount() has the same body as formatMoney() in src/billing/format.ts
HIGH src/billing/invoice.ts:6
Called an API that does not exist
computeDueDateFromTerms() is called but never defined or imported
HIGH src/billing/tax.ts:6
Swallowed an error
empty catch block
4 findings · 2 blocking · 0 style comments