Skip to content
This repository was archived by the owner on Jan 5, 2023. It is now read-only.

I followed rule of 3. #418

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

AhmadKazimi
Copy link

@AhmadKazimi AhmadKazimi commented Apr 24, 2022

I found that there's redundant code when it comes to the below code block

lifecycleScope.launch {
repeatOnLifecycle(Lifecycle.State.STARTED) {}
}

so I just replace it with ext function called **collectLifecycleFlow()**
for AppCompatActivity and Fragment 

…Lifecycle(Lifecycle.State.STARTED) {}} I found many usage in same level and created ext. fun to handle flow collect operation.
@google-cla
Copy link

google-cla bot commented Apr 24, 2022

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

For more information, open the CLA check for this pull request.

# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant