Skip to content
New issue

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

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

Already on GitHub? # to your account

fix: avoid scrolling sample code; fix test description; add () to functions calls #4080

Merged
merged 1 commit into from
Apr 3, 2024

Conversation

PetrakovichVictoria
Copy link
Contributor

No description provided.

@qwwdfsad qwwdfsad self-requested a review April 3, 2024 10:24
@qwwdfsad qwwdfsad merged commit d106ac7 into master Apr 3, 2024
1 check passed
@qwwdfsad qwwdfsad deleted the minor-fixes-in-coroutines-and-channels-tutorial branch April 3, 2024 10:25
Comment on lines +95 to +96
service: GitHubService,
req: RequestData
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is the indentation here correct?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, this is a copy-paste issue from a file with code to the example area in the md file. Fixed it here https://github.com/Kotlin/kotlinx.coroutines/pull/4085/files

knisht pushed a commit to JetBrains/intellij-deps-kotlinx.coroutines that referenced this pull request Apr 15, 2024
mikegr pushed a commit to mikegr/kotlinx.coroutines that referenced this pull request Apr 17, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants