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

fixed prototype comparison #40

Closed
wants to merge 4 commits into from
Closed

fixed prototype comparison #40

wants to merge 4 commits into from

Conversation

vruivo
Copy link
Contributor

@vruivo vruivo commented May 10, 2016

also added RegExp comparison and some tests

@vruivo vruivo closed this May 10, 2016
@vruivo vruivo reopened this May 10, 2016
@ghost
Copy link

ghost commented Oct 7, 2016

@vruivo I found a better alternative here: https://github.com/zubuzon/kewlr. Seems to support this

@ljharb
Copy link
Member

ljharb commented Jul 31, 2019

@vruivo if you can resurrect this repository and check the "allow edits" box, we can get this in.

@ljharb
Copy link
Member

ljharb commented Jul 31, 2019

Added a variant of this RegExp fix, and the tests, in b8c179c.

@ljharb ljharb added the needs "allow edits" Please check "allow edits" in the right hand side of the PR. label Jul 31, 2019
@ljharb
Copy link
Member

ljharb commented Dec 1, 2019

I believe b8c179c and 43b1eef (and probably a few others) have covered this,

@vruivo if you can check "allow edits" i can update the PR and make sure it's properly merged :-)

@BridgeAR

This comment has been minimized.

ljharb added a commit that referenced this pull request Dec 4, 2019
@ljharb
Copy link
Member

ljharb commented Dec 5, 2019

Resolved in 1eac46b

@ljharb ljharb closed this Oct 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs "allow edits" Please check "allow edits" in the right hand side of the PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants