search

okwolf / hyperapp-logger

Log Hyperapp state updates and action information to the console.
MIT License
47 stars 5 forks source link

Hooks API Refactor #6

Closed okwolf closed 7 years ago

okwolf commented 7 years ago

Preparing for https://github.com/hyperapp/hyperapp/pull/385 by using hooks instead of events. Not being passed the current state anywhere in the curried action functions available resulted in some code indirection. It appears to work, but I'm thinking I might want to make this and some other useful HOF for hooks shared code.

codecov[bot] commented 7 years ago

Codecov Report

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

Impacted file tree graph

@@          Coverage Diff          @@
##           master     #6   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           1      1           
  Lines          18     19    +1     
  Branches        3      3           
=====================================
+ Hits           18     19    +1
Impacted Files Coverage Δ
src/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 c1936bf...86a8abf. Read the comment docs.

okwolf commented 7 years ago

Closing in favor of #7