search

loong / go-concurrency-exercises

Hands on exercises with real-life examples to study and practice Go concurrency patterns. Test-cases are provided to verify your answers.
Other
1.11k stars 378 forks source link

Revamping repo, suggestions welcome! #23

Open loong opened 1 year ago

loong commented 1 year ago

Hello everyone,

I'm grateful to come back to this project and see we have passed 500 stars! This project was originally made for fun to show at a Go Meetup in Hsinchu, Taiwan (where only 2 other people showed up).

This repo def deserves some revamping! Most of this was made back in 2016, and Go as a language has evolved since then!

Let me know if you have any wishes! I keep a list of ideas below.

Ideas

dangdennis commented 1 year ago

This has been a great refresher. Thanks @loong.

Go modules is a good next step, and fixing any lint errors.

In 0-limit-crawler, there are a couple warnings.

  1. use time.Since(start)
  2. according to testinggoroutine, call to t.FailNow should be in the main test goroutine. i'm unsure how to fix though.

In 1-producer-consumer, yeah it'd be nice for someone revisiting this to simply run go run main.go instead of remembering to run go run main.go mockstream.go.

I'll report any more possible changes when I'm done.