Skip to content

Conversation

@cnaples79
Copy link

@cnaples79 cnaples79 commented Dec 21, 2025

Summary

  • format currency output with thousands separators while preserving sign
  • add unit tests covering positive and negative amounts

Rationale

Changes

  • update FormatCurrency to insert commas and keep the sign separate
  • add formatter tests for comma formatting on positive/negative values

Fixes #3

@tryonlinux
Copy link
Owner

Thanks for the PR! However it looks like we both were working on the comma thing (#4 ) at almost the same time and I already merged the comma fix in about 20 mins before you created this PR! Sorry for the duplicative work! Didn't expect the issue to be picked up so quickly by someone else.

@tryonlinux tryonlinux closed this Dec 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add commas to formatting

2 participants