Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat: support modifiers proposal #28

Merged
merged 2 commits into from
Jun 6, 2023
Merged

Conversation

liuxingbaoyu
Copy link
Contributor

if(a.type == "quantifier" && b.type == "quantifier" && a.symbol == null && b.symbol == "+"){
b.symbol = undefined;
}

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Here is the bug fix, when I updated regjsparser to 0.8.4 it failed the test.

@nicolo-ribaudo
Copy link
Contributor

cc @bnjmnt4n if you have time for a review!

@nicolo-ribaudo
Copy link
Contributor

Ping @bnjmnt4n :)

@bnjmnt4n bnjmnt4n merged commit ac8b07e into bnjmnt4n:main Jun 6, 2023
@bnjmnt4n
Copy link
Owner

bnjmnt4n commented Jun 6, 2023

Thanks for the PR and sorry for the delay! Will push a release asap. @nicolo-ribaudo I can add you to the repo + npm package if you'd like. Thanks as always for the awesome work on babel!

bnjmnt4n added a commit that referenced this pull request Jun 6, 2023
- Add support for modifiers proposal (#28)
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants