Skip to content

Conversation

gagarine
Copy link
Contributor

It's kind of strange to not use the passed variable in the example.

@wouterj
Copy link
Member

wouterj commented Mar 22, 2013

👍, but it should get merged into the 2.0 branch (not the master branch). @weaverryan will do it this time, but it would be easier if you choose the correct branch the next time. Read the contributors guide for more info.

weaverryan added a commit that referenced this pull request Mar 30, 2013
@weaverryan
Copy link
Member

Hi Simon!

I like what you've done here - the example is now much more complete. I've patched the commit into the 2.0 branch at sha: 58d3c57 - we'll merge it up through the rest of the branches from there. I also discovered some bad English here (not introduced with this fix), which was fixed at sha: aa7fde2

Thanks!

@weaverryan weaverryan closed this Mar 30, 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