Skip to content
This repository has been archived by the owner. It is now read-only.

Consistency fixes #30

Merged
merged 3 commits into from Jul 3, 2017

Conversation

Projects
None yet
2 participants
@c-w
Copy link
Member

commented Jul 3, 2017

  1. Always use catch(reject) for simple promise error handling.
  2. Whitespace fixes
  3. When rejecting a promise, inline the return.

No functional change.

@c-w c-w requested a review from Smarker Jul 3, 2017

@Smarker

Smarker approved these changes Jul 3, 2017

Copy link
Contributor

left a comment

LGTM

@c-w c-w merged commit ba47433 into master Jul 3, 2017

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@c-w c-w deleted the consistency-fixes branch Jul 3, 2017

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
You can’t perform that action at this time.