X Tutup
Skip to content

fix: allow d and v flag in regExpLiteral builder#17495

Merged
JLHwung merged 1 commit intobabel:mainfrom
JLHwung:allow-more-regexp-flags-in-regexp-literal-constructor
Aug 29, 2025
Merged

fix: allow d and v flag in regExpLiteral builder#17495
JLHwung merged 1 commit intobabel:mainfrom
JLHwung:allow-more-regexp-flags-in-regexp-literal-constructor

Conversation

@JLHwung
Copy link
Contributor

@JLHwung JLHwung commented Aug 28, 2025

Q                       A
Fixed Issues? Babel 8 rejects d and v flag in t.regExpLiteral builder
Patch: Bug Fix? Yes
Major: Breaking Change?
Minor: New Feature?
Tests Added + Pass? Yes
Documentation PR Link
Any Dependency Changes?
License MIT

This issue does not affect Babel 7 as Babel 7 does not enforce the flags input. Spot this issue when working on #17494.

@JLHwung JLHwung added the PR: Bug Fix (next major) 🐛 A type of pull request used for our changelog categories for next major release label Aug 28, 2025
@pkg-pr-new
Copy link

pkg-pr-new bot commented Aug 28, 2025

Open in StackBlitz

commit: 16fb9fd

@babel-bot
Copy link
Collaborator

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

@JLHwung JLHwung merged commit 7f57d3a into babel:main Aug 29, 2025
76 of 77 checks passed
@JLHwung JLHwung deleted the allow-more-regexp-flags-in-regexp-literal-constructor branch August 29, 2025 12:27
This was referenced Oct 21, 2025
This was referenced Nov 26, 2025
@github-actions github-actions bot added the outdated A closed issue/PR that is archived due to age. Recommended to make a new issue label Nov 29, 2025
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 29, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

outdated A closed issue/PR that is archived due to age. Recommended to make a new issue PR: Bug Fix (next major) 🐛 A type of pull request used for our changelog categories for next major release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

X Tutup