Skip to content

chore: avoid outdated regex in readNumber#17837

Merged
nicolo-ribaudo merged 1 commit intobabel:mainfrom
JLHwung:read-number-cleanup
Feb 27, 2026
Merged

chore: avoid outdated regex in readNumber#17837
nicolo-ribaudo merged 1 commit intobabel:mainfrom
JLHwung:read-number-cleanup

Conversation

@JLHwung
Copy link
Copy Markdown
Contributor

@JLHwung JLHwung commented Feb 27, 2026

Q                       A
Fixed Issues?
Patch: Bug Fix?
Major: Breaking Change?
Minor: New Feature?
Tests Added + Pass? Yes
Documentation PR Link
Any Dependency Changes?
License MIT

The m in the regex [_mn] is no longer relevant. And it turns out we don't have to use regex at all with the replaceAll method.

@babel-bot
Copy link
Copy Markdown
Collaborator

Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/61039

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new bot commented Feb 27, 2026

Open in StackBlitz

commit: 287f195

@nicolo-ribaudo nicolo-ribaudo merged commit eb64ddf into babel:main Feb 27, 2026
55 checks passed
@nicolo-ribaudo nicolo-ribaudo deleted the read-number-cleanup branch February 27, 2026 16:17
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.

4 participants