Skip to content
This repository has been archived by the owner on Nov 12, 2024. It is now read-only.

rememberStateWithLifecycle utility #900

Merged
merged 1 commit into from
Feb 19, 2022
Merged

rememberStateWithLifecycle utility #900

merged 1 commit into from
Feb 19, 2022

Conversation

yschimke
Copy link
Contributor

Minor cleanup and simplification of initial value which exists from StateFlow.

I was using your implementation, but wanted to avoid passing in an extra initial value,
and figured worth pushing back to here. No problem if not desired, but I'd love to here your
take if so.

Minor cleanup and simplification of initial value
which exists from StateFlow.
Copy link
Owner

@chrisbanes chrisbanes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@chrisbanes chrisbanes enabled auto-merge (squash) February 19, 2022 09:08
@chrisbanes chrisbanes disabled auto-merge February 19, 2022 09:09
@chrisbanes chrisbanes merged commit bce20b5 into chrisbanes:main Feb 19, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants