Skip to content
This repository was archived by the owner on May 18, 2026. It is now read-only.

implemented redux-logger given an environmental variable - #61

Merged
thejmazz merged 1 commit into
masterfrom
logger_updated
Jul 7, 2017
Merged

implemented redux-logger given an environmental variable#61
thejmazz merged 1 commit into
masterfrom
logger_updated

Conversation

@tiagofilipe12

Copy link
Copy Markdown
Member

In this PR implemented a check for an environmental variable that changes the middlewares within lib/watermill.js. If REDUX_LOGGER=1 in environment the pipeline will execute redux-logger.
Also redux-logger package was updated in this PR.

@codecov-io

Copy link
Copy Markdown

Codecov Report

Merging #61 into master will increase coverage by 0.01%.
The diff coverage is 85.71%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #61      +/-   ##
==========================================
+ Coverage   80.64%   80.66%   +0.01%     
==========================================
  Files          36       36              
  Lines         806      812       +6     
  Branches       99      100       +1     
==========================================
+ Hits          650      655       +5     
- Misses        156      157       +1
Impacted Files Coverage 螖
lib/watermill.js 94.73% <85.71%> (-5.27%) 猬囷笍

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 b3e1588...9fbcb32. Read the comment docs.

@thejmazz
thejmazz merged commit 23ef9de into master Jul 7, 2017
@tiagofilipe12 tiagofilipe12 mentioned this pull request Jul 7, 2017
@tiagofilipe12
tiagofilipe12 deleted the logger_updated branch July 7, 2017 21:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants