New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Article: "Do Not. Subscribe." #10

Open

RLittlesII opened this issue Feb 5, 2021 · 0 comments

Owner

RLittlesII commented Feb 5, 2021 •

edited

Stop using Do. Use Subscribe and compose observables like a human!

Do runs things out of process
Affecting state in Do is risky
The documentation states its suitable for logging, would you put business logic in logging?!
Multiple Do means lack of Composition
Do instead of Subscribe is a smell
Do can't surface notifications, you have to subscribe anyway

RLittlesII changed the title ~~Article: "Do Not. Subscribe"~~ Article: "Do Not. Subscribe."

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment