Closed the-dr-lazy closed 5 years ago
@LouizFC, @rlesniak - If it's possible please review this PR.
Merging #84 into next will not change coverage. The diff coverage is
100%
.
@@ Coverage Diff @@
## next #84 +/- ##
===================================
Coverage 100% 100%
===================================
Files 9 9
Lines 47 47
Branches 11 11
===================================
Hits 47 47
Impacted Files | Coverage Δ | |
---|---|---|
src/index.ts | 100% <ø> (ø) |
:arrow_up: |
src/create-reducer.ts | 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 452c522...6dc276b. Read the comment docs.
Merging #84 into next will not change coverage. The diff coverage is
100%
.
@@ Coverage Diff @@
## next #84 +/- ##
===================================
Coverage 100% 100%
===================================
Files 9 9
Lines 47 47
Branches 11 11
===================================
Hits 47 47
Impacted Files | Coverage Δ | |
---|---|---|
src/index.ts | 100% <ø> (ø) |
:arrow_up: |
src/create-reducer.ts | 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 452c522...6dc276b. Read the comment docs.
As a conclusion in #58 and #55; This PR make using
DeepImmutable
optional in handler's input/prev state argument.Public API Changes
Rationally, because using
DeepImmutable
is optional, public API should export immutability type helpers.Adds
Immutable
DeepImmutable
DeepImmutableArray
DeepImmutableMap
DeepImmutableObject