search

dwyl / redis-connection

:zap: Single Redis Connection that can be used anywhere in your node.js app and closed once (e.g in tests)
45 stars 12 forks source link

adds event listener for uncaughtException to report connection error #39

Closed nelsonic closed 7 years ago

nelsonic commented 7 years ago

fixes #38

codecov[bot] commented 7 years ago

Codecov Report

Merging #39 into master will not change coverage. The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff          @@
##           master    #39   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           1      1           
  Lines          28     38   +10     
  Branches        5      6    +1     
=====================================
+ Hits           28     38   +10
Impacted Files Coverage Δ
index.js 100% <100%> (ø) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update ba348e0...2f68ec6. Read the comment docs.

nelsonic commented 7 years ago

@iteles not "urgent" but good to get in for the work I'm doing. thanks! 👍