Skip to content

Document structured concurrency #4433

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

Draft
wants to merge 5 commits into
base: develop
Choose a base branch
from

Conversation

dkhalanskyjb
Copy link
Collaborator

@dkhalanskyjb dkhalanskyjb commented May 9, 2025

Draft for now, because the documentation of some other functions should be updated accordingly:

  • The remaining entries in CoroutineScope.kt.
  • supervisorScope
  • Builders.common.kt: launch, async, and the other coroutine builders should document their structured concurrency behaviors.

but reviewing the changes should already be okay (I'll introduce further changes in separate commits).

cc @LouisCAD, given your real-world experience and interest in the documentation, your comments on this would be greatly appreciated!

@dkhalanskyjb dkhalanskyjb requested a review from qwwdfsad May 9, 2025 12:08
@dkhalanskyjb dkhalanskyjb force-pushed the dkhalanskyjb/fix-structured-concurrency-docs branch from ea4a81c to 4ad4272 Compare May 21, 2025 09:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant