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

chore: Fix a couple broken book links #172

Merged
merged 1 commit into from
Jun 23, 2019

Conversation

miller-time
Copy link
Contributor

While I was fixing these, I figured maybe it's good to just use a consistent URL for book links: https://doc.rust-lang.org/book/

Is there a plan for what to do with this old 1.4.0 example that doesn't exist in the current version of the book? Perhaps at least include a disclaimer when recommending that folks check it out?

For this exercise check out the [Dining Philosophers example](https://doc.rust-lang.org/1.4.0/book/dining-philosophers.html) and the chapter [Concurrency](https://doc.rust-lang.org/book/ch16-01-threads.html) of the Rust Book.

@miller-time
Copy link
Contributor Author

@komaeda what would the <scope> be for this pull request's title?

@komaeda
Copy link
Contributor

komaeda commented Jun 9, 2019

chore: Fix a couple broken book links

@bors
Copy link
Contributor

bors commented Jun 9, 2019

☔ The latest upstream changes (presumably #174) made this pull request unmergeable. Please resolve the merge conflicts.

@bors bors added the S-waiting-on-author Status: Waiting on issue/PR author label Jun 9, 2019
@miller-time miller-time changed the title fix a couple broken book links chore: Fix a couple broken book links Jun 9, 2019
@komaeda
Copy link
Contributor

komaeda commented Jun 23, 2019

@bors: r+

@bors
Copy link
Contributor

bors commented Jun 23, 2019

📌 Commit 63c133e has been approved by komaeda

@bors bors added S-waiting-on-bors and removed S-waiting-on-author Status: Waiting on issue/PR author labels Jun 23, 2019
@bors
Copy link
Contributor

bors commented Jun 23, 2019

⌛ Testing commit 63c133e with merge aaad60d...

bors added a commit that referenced this pull request Jun 23, 2019
chore: Fix a couple broken book links

While I was fixing these, I figured maybe it's good to just use a consistent URL for book links: https://doc.rust-lang.org/book/

Is there a plan for what to do with this old 1.4.0 example that doesn't exist in the current version of the book? Perhaps at least include a disclaimer when recommending that folks check it out?

https://github.com/rust-lang/rustlings/blob/34e31232dfddde284a341c9609b33cd27d9d5724/exercises/threads/README.md#L1
@bors
Copy link
Contributor

bors commented Jun 23, 2019

☀️ Test successful - checks-travis
Approved by: komaeda
Pushing aaad60d to master...

@bors bors merged commit 63c133e into rust-lang:master Jun 23, 2019
ppp3 pushed a commit to ppp3/rustlings that referenced this pull request May 23, 2022
chore: Fix a couple broken book links

While I was fixing these, I figured maybe it's good to just use a consistent URL for book links: https://doc.rust-lang.org/book/

Is there a plan for what to do with this old 1.4.0 example that doesn't exist in the current version of the book? Perhaps at least include a disclaimer when recommending that folks check it out?

https://github.com/rust-lang/rustlings/blob/34e31232dfddde284a341c9609b33cd27d9d5724/exercises/threads/README.md#L1
dmoore04 pushed a commit to dmoore04/rustlings that referenced this pull request Sep 11, 2022
chore: Fix a couple broken book links

While I was fixing these, I figured maybe it's good to just use a consistent URL for book links: https://doc.rust-lang.org/book/

Is there a plan for what to do with this old 1.4.0 example that doesn't exist in the current version of the book? Perhaps at least include a disclaimer when recommending that folks check it out?

https://github.com/rust-lang/rustlings/blob/4a6c4754c684a2c7ee757e8bccf9955f92ef6dab/exercises/threads/README.md#L1
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.

3 participants