Skip to content

Conversation

@hoc081098
Copy link
Owner

@hoc081098 hoc081098 commented Aug 2, 2022

.

@hoc081098 hoc081098 changed the title refactor(flowredux): SideEffect receive a StateFlow instead of a `G… refactor(flowredux): SideEffect receive a StateFlow instead of a GetState Aug 2, 2022
@codecov
Copy link

codecov bot commented Aug 2, 2022

Codecov Report

Merging #7 (0db01e8) into master (ea90751) will decrease coverage by 0.08%.
The diff coverage is 16.66%.

@@            Coverage Diff            @@
##           master      #7      +/-   ##
=========================================
- Coverage    3.50%   3.41%   -0.09%     
=========================================
  Files          47      47              
  Lines        1083    1082       -1     
  Branches      143     143              
=========================================
- Hits           38      37       -1     
  Misses       1038    1038              
  Partials        7       7              
Impacted Files Coverage Δ
...search_kmm/presentation/GithubSearchSideEffects.kt 0.00% <0.00%> (ø)
...n/com/hoc081098/flowredux/DefaultFlowReduxStore.kt 96.96% <100.00%> (-0.09%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us.

@hoc081098 hoc081098 linked an issue Aug 2, 2022 that may be closed by this pull request
@hoc081098 hoc081098 merged commit 4303d1d into master Aug 2, 2022
@hoc081098 hoc081098 deleted the flowredux/stateflow branch August 2, 2022 04:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

SideEffects: GetState to StateFlow

2 participants