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

Simplify the event stream #25

Closed

lu-zero wants to merge 2 commits into master from simpler-stream

Collaborator

lu-zero commented Dec 30, 2024

I wished I could trade futures_util for futures_core but for_each is still nicer.


          Simplify the event stream

4beca26

lu-zero requested a review from nappa85

December 30, 2024 12:01


          Avoid map(await { }

39e351e

The result of the map was not awaited.

Collaborator Author

lu-zero commented Dec 30, 2024

#26 goes in the opposite direction and probably is better to show how to deal with streams and futures (and should perform better).

lu-zero closed this

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

Labels

None yet