search

TritonDataCenter / node-verror

Rich JavaScript errors
MIT License
1.18k stars 61 forks source link

add `findCauseByName` documentation #43

Closed DonutEspresso closed 8 years ago

DonutEspresso commented 8 years ago

Sorry for the extra noise in the PR, looks I've got a plugin that strips whitespace. Happy to add the whitespace back in if it's a problem.

I also noticed in the contribution guidelines that cr.joyent.us is preferred over github PRs - if that's still the case, let me know and I can refile.

davepacheco commented 8 years ago

This looks good. Would you mind posting it to cr.joyent.us? Let me know if you have any trouble with it.

DonutEspresso commented 8 years ago

OK, I think I migrated it over correctly - let me know if it needs any other touchups: https://cr.joyent.us/#/c/325/

davepacheco commented 8 years ago

I just edited the commit message to match our convention. If you're cool with it, I'll integrate the change.

DonutEspresso commented 8 years ago

Works for me. Thanks!

davepacheco commented 8 years ago

Integrated. Thanks for doing this!