README example typo (properly fixed) #402

Merged
merged 2 commits into from Jan 13, 2013

Projects

None yet

2 participants

@banks
Contributor
banks commented Dec 30, 2012

Pull request 401 attempted to fix an error but actually introduced one. This corrects it and removes the confusion.

The current README would execute correctly but is very confusing since it shows a callback with context and options args when the helper has no context parameter. It then actually uses context.fn() which works but is very confusing given the convention of calling this hash options everywhere else.

@kpdecker kpdecker merged commit 76882ff into wycats:master Jan 13, 2013

1 check failed

default The Travis build failed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment