-
-
Notifications
You must be signed in to change notification settings - Fork 406
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Merged by Bors] - Refactor RegExp
constructor methods
#2049
Conversation
Test262 conformance changesVM implementation
Fixed tests (8):
|
Codecov Report
@@ Coverage Diff @@
## main #2049 +/- ##
=======================================
Coverage 44.16% 44.17%
=======================================
Files 212 212
Lines 18812 18815 +3
=======================================
+ Hits 8309 8312 +3
Misses 10503 10503
Continue to review full report at Codecov.
|
Benchmark for 78a5d16Click to view benchmark
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me! :)
bors r+ |
This PR was included in a batch that was canceled, it will be automatically retried |
Pull request successfully merged into main. Build succeeded: |
RegExp
constructor methodsRegExp
constructor methods
This Pull Request changes the following:
RegExp
constructor methods to match the spec and be more efficient