Skip to content
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

String.prototype.matchAll fixed on Firefox 73 #1559

Merged
merged 3 commits into from Dec 8, 2019

Conversation

afmenez
Copy link
Contributor

@afmenez afmenez commented Dec 8, 2019

String.prototype.matchAll now throws on non-global regEx.

@zloirock zloirock merged commit 14a96e6 into compat-table:gh-pages Dec 8, 2019
@afmenez afmenez deleted the ff73matchall branch December 8, 2019 19:19
@chicoxyzzy
Copy link
Member

The first two commits in this PR are not related to String.prototype.matchAll

@afmenez
Copy link
Contributor Author

afmenez commented Dec 8, 2019

Yep, and they are not even mine. Don't know how they appeared here.

@zloirock
Copy link
Collaborator

zloirock commented Dec 8, 2019

Seems @ljharb forced pushed to gh-pages :-D

@ljharb
Copy link
Member

ljharb commented Dec 8, 2019

indeed; those commits shouldn't be on master.

@zloirock
Copy link
Collaborator

zloirock commented Dec 8, 2019

They should be.

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.

None yet

4 participants