Skip to content
This repository has been archived by the owner on Sep 27, 2022. It is now read-only.

Fix example in README #32

Merged
merged 1 commit into from Nov 8, 2016
Merged

Fix example in README #32

merged 1 commit into from Nov 8, 2016

Conversation

mattbierner
Copy link
Contributor

I think there are a few small errors in the example from the README:

  • Use correct package name.
  • Use built-in filter.
  • error is actually an object { 'stderr': '', error: null}.

@mattbierner
Fix example in README
7fed6d5 
I think there are a few small errors in the example from the README:

* Use correct package name.
* Use built-in filter.
* `error` is actually an object `{ 'stderr': '', error: null}`.
@coveralls
Copy link

coveralls commented Nov 8, 2016 •
edited

Coverage Status

Coverage remained the same at 90.845% when pulling 7fed6d5 on mattbierner:patch-1 into a17c23a on jwoos:master.

@jwoos
Copy link
Owner

jwoos commented Nov 8, 2016

Thanks for this!
I knew I must have missed some stuff in the README

@jwoos jwoos merged commit 1084a97 into jwoos:master Nov 8, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@mattbierner @coveralls @jwoos