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

#1056 Fixed grammar for nested raw blocks #1060

Merged
merged 4 commits into from
Jul 20, 2015
Merged

#1056 Fixed grammar for nested raw blocks #1060

merged 4 commits into from
Jul 20, 2015

Conversation

ericbn
Copy link
Contributor

@ericbn ericbn commented Jul 15, 2015

No description provided.

this.popState();
return 'END_RAW_BLOCK';
if (this.conditionStack[this.conditionStack.length-1] === 'raw') {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should be using this.topState() here, but it currently returns the second top instead of the first top. Opened an issue about it at zaach/jison#291

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This comment should be a comment in the source.

@kpdecker
Copy link
Collaborator

Looks good, just want to get that comment in the source and then I'll be glad to merge.

@kpdecker kpdecker mentioned this pull request Jul 19, 2015
@ericbn
Copy link
Contributor Author

ericbn commented Jul 19, 2015

Cool. Just added the comment to the source.

kpdecker added a commit that referenced this pull request Jul 20, 2015
#1056 Fixed grammar for nested raw blocks
@kpdecker kpdecker merged commit c83c363 into handlebars-lang:master Jul 20, 2015
@kpdecker
Copy link
Collaborator

Thanks!

@jmm
Copy link

jmm commented Jul 20, 2015

Thanks guys!

@kpdecker
Copy link
Collaborator

kpdecker commented Sep 1, 2015

Released in 4.0.0

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.

3 participants