comprehensive-rust icon indicating copy to clipboard operation
comprehensive-rust copied to clipboard

Add Arc and Mutex to Arc/Mutex example

Open sudonym1 opened this issue 2 years ago • 4 comments

sudonym1 avatar Jan 02 '23 17:01 sudonym1

Hey @sudonym1,

Thanks for the PR — I think you basically "solved" it by adding the code which I normally add when live-coding in class :smile: That is, your changes are better suited for a speaker note (#53) than the main text.

mgeisler avatar Jan 03 '23 16:01 mgeisler

@sudonym1 could you please recast this as an update to the speaker notes? Start by adding a <details> ... </details> block to the page and then explain things inside it. Tell the instructor what they should remember to show to the students.

mgeisler avatar Jan 05 '23 11:01 mgeisler

Sure, I can do that.

sudonym1 avatar Jan 05 '23 14:01 sudonym1

Sure, I can do that.

Awesome :smile: You simply need a <details> ... </details> element with some Markdown at the bottom of the page, that's it. There are a few examples of this in the course now: https://google.github.io/comprehensive-rust/welcome-day-1.html

mgeisler avatar Jan 06 '23 09:01 mgeisler

Let me close this PR for now — please open another with updated speaker notes if you find time :smile:

mgeisler avatar Jan 27 '23 17:01 mgeisler