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

Fix typos #2098

Closed
wants to merge 2 commits into from
Closed

Fix typos #2098

wants to merge 2 commits into from

Conversation

Inego
Copy link
Contributor

@Inego Inego commented Jun 21, 2020

No description provided.

@@ -428,19 +428,19 @@ fun main() = runBlocking {
supervisorScope {
val child = launch {
try {
println("Child is sleeping")
println("The child is sleeping")
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please also run ./gradlew knit to update actual .kt files

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It doesn't work, it says JDK_16 environment variable is not defined.

Copy link
Contributor Author

@Inego Inego Jun 27, 2020

Choose a reason for hiding this comment

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

It's really, really sad that I would have to install JDK 1.6 just to have some typos fixed :(

Copy link
Contributor

Choose a reason for hiding this comment

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

If the "Allow modifications from maintainers" checkbox is checked, maybe @qwwdfsad can do it?

@elizarov elizarov mentioned this pull request Jul 3, 2020
elizarov pushed a commit that referenced this pull request Jul 3, 2020
@elizarov
Copy link
Contributor

elizarov commented Jul 3, 2020

Thanks. I've run knit and merged manually via f4fb204 Btw, for the purposes of knit you can simply set JDK_16 to JDK_HOME. I'll see if I can patch the code to make it happen automatically.

@elizarov elizarov closed this Jul 3, 2020
@Inego
Copy link
Contributor Author

Inego commented Jul 3, 2020

Thank you, Roman!

@elizarov elizarov mentioned this pull request Jul 16, 2020
1 task
recheej pushed a commit to recheej/kotlinx.coroutines that referenced this pull request Dec 28, 2020
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.

None yet

4 participants