Skip to content
This repository has been archived by the owner on Jan 15, 2025. It is now read-only.

Fix rounding and to precision #128

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

amoshydra
Copy link
Contributor

@amoshydra amoshydra commented Jan 6, 2025

A work in-progress draft for fixing round and toPrecision in v22.

Todo:

  • create tests for toPrecision
  • create tests for round
  • fix precision
  • fix round

Summary

pending

Related to

#116, jessealama/proposal-decimal-polyfill#15

How is this tested?

See updated tests for round and toPrecision

@amoshydra
Copy link
Contributor Author

I have added more tests for both toPrecision and round that will also cover the issues reported in #116 and jessealama/proposal-decimal-polyfill#15.

The tests can be reviewed first.

  • create tests for toPrecision
  • create tests for round
  • fix precision
  • fix round

I will try to look into how the issue could be fixed in the next few days

@jessealama jessealama marked this pull request as ready for review January 15, 2025 08:00
@jessealama
Copy link
Owner

Could you please move this to jessealama/proposal-decimal-polyfill? This repo was supposed to be sunset in favor of proposal-decimal-polyfill.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants