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

Issue #332 provide a raw input syntax similar to comments {!...!} where ... #334

Merged
merged 10 commits into from
Dec 9, 2013

Conversation

jimmyhchan
Copy link
Contributor

...newlines, spaces and braces are kept. should also address #238 and #265

…..!} where newlines, spaces and braces are kept. should also address linkedin#238 and linkedin#265
@rragan
Copy link
Contributor

rragan commented Sep 11, 2013

This would also be useful if generating JSON objects to the output stream. No need to escape the braces.

@@ -1254,7 +1299,7 @@ var coreTests = [
name: "error: whitespaces between the opening brace and any of (#,?,@,^,+,%) is not allowed",
source: '{ # helper foo="bar" boo="boo" } {/helper}',
context: { "helper": function(chunk, context, bodies, params) { return chunk.write(params.boo + " " + params.foo); } },
error: 'Expected buffer, comment, partial, reference, section or special but "{" found. At line : 1, column : 1',
error: 'Expected buffer, comment, partial, raw, reference, section or special but "{" found. At line : 1, column : 1',
Copy link
Contributor

Choose a reason for hiding this comment

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

Did you update the error message in the grammar? I don't see this update.

@@ -1240,6 +1282,9 @@ var coreTests = [
}
]
},
/**
* CORE GRAMMER TESTS
Copy link

Choose a reason for hiding this comment

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

s/GRAMMER/GRAMMAR/

@brikis98
Copy link

brikis98 commented Oct 9, 2013

lgtm 👍

@jimmyhchan
Copy link
Contributor Author

to keep master clean we should not pull this until after the 2.1 milestones bugs are merged.

@prashn64
Copy link
Contributor

prashn64 commented Dec 8, 2013

I think it's time we pulled this in. Jimmy, can you update your branch?

@jimmyhchan
Copy link
Contributor Author

Updated.

Look: nothing changes in dist and package json stays at 2.2.3..... we'll do the bumps when we release the next version

prashn64 added a commit that referenced this pull request Dec 9, 2013
Issue #332 provide a raw input syntax similar to comments {!...!} where newlines, spaces and braces are kept.
@prashn64 prashn64 merged commit b5a2a6e into linkedin:master Dec 9, 2013
@jimmyhchan jimmyhchan deleted the raw branch December 13, 2013 06:58
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.

5 participants