Skip to content

fix(BigDecimal.normalize): return self if already normalized#6264

Open
Hoishin wants to merge 2 commits into
Effect-TS:mainfrom
Hoishin:fix-bigdecimal-normalize
Open

fix(BigDecimal.normalize): return self if already normalized#6264
Hoishin wants to merge 2 commits into
Effect-TS:mainfrom
Hoishin:fix-bigdecimal-normalize

Conversation

@Hoishin

@Hoishin Hoishin commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

Type

  • Refactor
  • Feature
  • Bug Fix
  • Optimization
  • Documentation Update

Description

Currently normalize always normalize the digits whether it's already normalized or not. It fails with stricter assertions in Node 24.

Related

@Hoishin Hoishin requested a review from mikearnaldi as a code owner June 8, 2026 17:11
@github-project-automation github-project-automation Bot moved this to Discussion Ongoing in PR Backlog Jun 8, 2026
@changeset-bot

changeset-bot Bot commented Jun 8, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 8f40d56

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
effect Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@Hoishin Hoishin mentioned this pull request Jun 8, 2026
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Discussion Ongoing

Development

Successfully merging this pull request may close these issues.

1 participant