54.1 Join (dependencies)

[mgeisler]

Discussed in #1317

^{Originally posted by njr0 October 6, 2023}
I had to add these dependencies to may Cargo.toml to make it work. As with the last few suggestions, I think it will help to includes these in the speaker notes or main text, especially since this can't be run in the playground. (And who wants to be in a playground anyway?)

[dependencies]
tokio = { version = "1.32.0", features = ["full"] }
reqwest = { version = "0.11.22", features = ["blocking", "rustls-tls"] }
futures = "0.3"
anyhow = "1.0"

[0scvr]

Is this the section you are referring to src/async/control-flow/join.md ?

I wouldn't mind working on this.

[mgeisler]

Is this the section you are referring to src/async/control-flow/join.md ?

Yeah, that sounds right, @njr0 can confirm and give more input here.

I wouldn't mind working on this.

Awesome, thanks a bunch!

[mgeisler]

In general, @njr0 had a ton of excellent feedback and I probably only captured some of it in the issues. It's all available in the discussions.

[djmitche]

There are some instructions for running the course here:

https://github.com/google/comprehensive-rust/blob/main/src/running-the-course/course-structure.md#concurrency-in-rust

Perhaps those should be linked in some/all of the speaker notes for the concurrency section?

[0scvr]

There are some instructions for running the course here:

https://github.com/google/comprehensive-rust/blob/main/src/running-the-course/course-structure.md#concurrency-in-rust

Perhaps those should be linked in some/all of the speaker notes for the concurrency section?

In that case, I think

comprehensive-rust/src/running-the-course/course-structure.md

Lines 57 to 62 in 77c55ef

	```shell
	cargo init concurrency
	cd concurrency
	cargo add tokio --features full
	cargo run
	```

is incomplete. Tokio isn't the only crate used. e.g. the futures, anyhow and reqwest crates are used here https://google.github.io/comprehensive-rust/async/control-flow/join.html