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.

Sign up for GitHub

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

Jump to bottom

Add support for lookbehinds #89

Merged
merged 1 commit into from
Jan 21, 2018
Merged

Add support for lookbehinds #89

merged 1 commit into from
Jan 21, 2018

Conversation

adrianheine
Copy link
Contributor

@jviereck
Copy link
Owner

Thx @adrianheine for the PR. Can you update the README.md as well to show the lookbehind exists?

@adrianheine
Copy link
Contributor Author

Sure, done that and also updated the AST documentation.

@jviereck jviereck merged commit 00f49a1 into jviereck:gh-pages Jan 21, 2018
@jviereck
Copy link
Owner

Thanks!

@adrianheine adrianheine deleted the lookbehind branch January 26, 2018 12:39
@ewanharris
Copy link

@jviereck Hey there, is it possible for this to be published to npm? It looks like this hasn't been published yet? Thanks!

@jviereck
Copy link
Owner

@ewanharris, thanks - I just published v0.4.0 to npm. Let me know if there is anything else.

bnjmnt4n pushed a commit to bnjmnt4n/regjsgen that referenced this pull request Oct 24, 2018
@adrianheine @bnjmnt4n
Add support for lookbehind assertions.
e707378 
Closes #11, #12.
Ref. https://github.com/tc39/proposal-regexp-lookbehind
Ref. jviereck/regjsparser#89
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@adrianheine @jviereck @ewanharris