Skip to content

Make matching regexes more permissive#13

Open
kgutwin wants to merge 1 commit intodanielevans:masterfrom
kgutwin:master
Open

Make matching regexes more permissive#13
kgutwin wants to merge 1 commit intodanielevans:masterfrom
kgutwin:master

Conversation

@kgutwin
Copy link
Copy Markdown

@kgutwin kgutwin commented Oct 11, 2018

Valid handlebars syntax such as:

{{#each model as |photo|}}
{{#if (eq params.photo_id) photo.id}}
{{else if (eq params.photo_id) "all"}}

was not being matched properly. This PR opens up several regexes so that any character except a close-brace following the initial token will be matched successfully. While this may still fail in certain circumstances, it's at least better for the majority of cases.

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.

1 participant