8000 Update readme.md by dariusk · Pull Request #8 · galaxykate/tracery · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
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 readme.md #8

Merged
merged 2 commits into from
Nov 28, 2015
Merged

Update readme.md #8

merged 2 commits into from
Nov 28, 2015

Conversation

dariusk
Copy link
Contributor
@dariusk dariusk commented Nov 28, 2015

I had copy/pasted the example grammar and was getting tons of errors -- I thought it was my code but it turns out the grammar was using outdated syntax for saving data. Updating that here with something from the tutorials that works in both the browser and Node.js libs.

I also added a link to @v21's Node lib. I have a pending pull request over there that would make it very clear how to install the library via npm.

I had copy/pasted the example grammar and was getting tons of errors -- I thought it was my code but it turns out the grammar was using outdated syntax for saving data. Updating that here with something from the tutorials that works in both the browser and Node.js libs.

I also added a link to @v21's Node lib. I have a [pending pull request](v21#1) over there that would make it very clear how to install the library via `npm`.
Also removing `.plural` from the modifier examples since that's outdated syntax, replacing with `.s`
galaxykate pushed a commit that referenced this pull request Nov 28, 2015
@galaxykate galaxykate merged commit fc98c0b into galaxykate:master Nov 28, 2015
@v21
Copy link
v21 commented Nov 30, 2015

It's worth noting that I've not updated the npm library to new Tracery yet - that is not quite trivial to do, but is on my list (as then I can upgrade CBDQ)

@galaxykate
Copy link
Owner

If there's anything I can do to make porting eaiser, too, lmk. I don't know
what the additional constraints are, but if I know I can make sure I
support them going forward.
On Nov 30, 2015 4:04 PM, "George Buckenham" notifications@github.com
wrote:

It's worth noting that I've not updated the npm library to new Tracery yet

  • that is not quite trivial to do, but is on my list (as then I can upgrade
    CBDQ)


Reply to this email directly or view it on GitHub
#8 (comment).

@v21
Copy link
v21 commented Nov 30, 2015

Opened up a new issue to chat about this on: #9

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
0