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

Update parser.js to sync with latest dust.pegjs #320

Merged
merged 2 commits into from
Aug 1, 2013

Conversation

smfoote
Copy link
Contributor

@smfoote smfoote commented Jul 30, 2013

Some of the latest changes in dust.pegjs did not make it into parser.js. This has been update by running:

make parser

Some of the latest changes in dust.pegjs did not make it into parser.js. This has been update by running:

`make parser`
@@ -1,6 +1,6 @@
{
"name": "dustjs-linkedin",
"version": "2.0.0",
"version": "2.0.2",
Copy link
Contributor

Choose a reason for hiding this comment

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

what happen to 2.0.1?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm not sure why this didn't get updated, but there was a 2.0.1.

@jimmyhchan
Copy link
Contributor

looks good

@vybs
Copy link
Contributor

vybs commented Aug 1, 2013

lgtm!

@vybs
Copy link
Contributor

vybs commented Aug 1, 2013

@jimmyhchan you have adming rights to npm publish the 2.0.2 as well.

please update product spec before publishing and tag the new version
second also update the wiki : http://linkedin.github.io/dustjs/

jimmyhchan added a commit that referenced this pull request Aug 1, 2013
Update parser.js to sync with latest dust.pegjs
@jimmyhchan jimmyhchan merged commit b040e27 into linkedin:master Aug 1, 2013
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