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