Skip to content
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.

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

Add fn main to root examples #3971

Merged
merged 2 commits into from
Nov 12, 2024
Merged

Add fn main to root examples #3971

merged 2 commits into from
Nov 12, 2024

Conversation

madsmtm
Copy link
Member

@madsmtm madsmtm commented Oct 25, 2024

This is not strictly required, but makes the examples a bit easier to read understand (especially since the EventLoop really should be created inside fn main, and not in some random function potentially running on a random thread).

CC #3969

This is not strictly required, but makes the examples a bit easier to
read understand (especially since the `EventLoop` really _should_ be
created inside `fn main`, and not in some random function potentially
running on a random thread).
@madsmtm madsmtm added S - docs Awareness, docs, examples, etc. C - needs discussion Direction must be ironed out labels Oct 25, 2024
@kchibisov kchibisov merged commit 74958ec into master Nov 12, 2024
58 checks passed
@kchibisov kchibisov deleted the madsmtm/doc-add-main branch November 12, 2024 13:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C - needs discussion Direction must be ironed out S - docs Awareness, docs, examples, etc.
Development

Successfully merging this pull request may close these issues.

2 participants