Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[Babel 8] doc: Remove decimal from parser#2945

Merged
nicolo-ribaudo merged 2 commits into
babel:mainfrom
liuxingbaoyu:remove-decimal
Sep 6, 2024
Merged

[Babel 8] doc: Remove decimal from parser#2945
nicolo-ribaudo merged 2 commits into
babel:mainfrom
liuxingbaoyu:remove-decimal

Conversation

@liuxingbaoyu
Copy link
Copy Markdown
Member

@netlify
Copy link
Copy Markdown

netlify Bot commented Aug 31, 2024

👷 Deploy request for babel-next pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 00f8916

@netlify
Copy link
Copy Markdown

netlify Bot commented Aug 31, 2024

Deploy Preview for babel ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 00f8916
🔍 Latest deploy log https://app.netlify.com/sites/babel/deploys/66d94ca0a37ea80008184b0f
😎 Deploy Preview https://deploy-preview-2945--babel.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Comment thread docs/v8-migration.md

- Remove `decimal` plugin option [#16741](https://github.com/babel/babel/pull/16741)

**Migration**: Migrate your project to the latest proposal and remove the plugin from your config since the latest proposal doesn't have syntax anymore.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe add an example? 1.03m becomes new Decimal("1.03"), and decimal1 + decimal2 becomes decimal1.add(decimal2).

Copy link
Copy Markdown
Member

@nicolo-ribaudo nicolo-ribaudo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The parser table needs to be updated, but I'll do it in #2948 to avoid conflicts.

@nicolo-ribaudo nicolo-ribaudo merged commit b5d8f96 into babel:main Sep 6, 2024
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.

2 participants