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: update variables5.rs book link #351

Merged
merged 1 commit into from
Apr 12, 2020

Conversation

saurav-2104
Copy link
Contributor

@saurav-2104 saurav-2104 commented Apr 12, 2020

chore: update variables5.rs book link

@commit-lint
Copy link

commit-lint bot commented Apr 12, 2020

Chore

  • update variables5.rs book link (1b9d748)

Contributors

@saurav-2104

@saurav-2104 saurav-2104 force-pushed the master branch 2 times, most recently from 5f7006d to a9a2e71 Compare April 12, 2020 15:41
@saurav-2104 saurav-2104 changed the title chore: Update variables5.rs book link chore: update variables5.rs book link Apr 12, 2020
@saurav-2104
Copy link
Contributor Author

As a first time user, I was not able to figure out which book the exercise was talking about.

info.toml Outdated
@@ -52,7 +52,8 @@ because we want to assign a different typed value to an existing variable. Somet
you may also like to reuse existing variable names because you are just converting
values to different types like in this exercise.
Fortunately Rust has a powerful solution to this problem: 'Shadowing'!
You can read more about 'Shadowing' in the book's section 'Variables and Mutability'.
You can read more about 'Shadowing' in the book's section 'Variables and Mutability':
https://doc.rust-lang.org/book/ch03-01-variables-and-mutability.html?highlight=variables#shadowing.
Copy link
Contributor

Choose a reason for hiding this comment

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

Hm, do we want the highlight query param?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh, my bad. I think I copied my searched results from browser. Removed it.

info.toml Outdated
@@ -52,7 +52,8 @@ because we want to assign a different typed value to an existing variable. Somet
you may also like to reuse existing variable names because you are just converting
values to different types like in this exercise.
Fortunately Rust has a powerful solution to this problem: 'Shadowing'!
You can read more about 'Shadowing' in the book's section 'Variables and Mutability'.
You can read more about 'Shadowing' in the book's section 'Variables and Mutability':
https://doc.rust-lang.org/book/ch03-01-variables-and-mutability.html#shadowing.
Copy link
Contributor

Choose a reason for hiding this comment

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

One last minor nit: in my terminal, the final dot messes with the link, so it ends up not jumping to the "Shadowing" section.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Removed "."
Also, cargo updated the Cargo.lock as well. At this point, I am not sure if it is unwanted.

chore: update variables5.rs book link

chore: update variables5.rs book link
@shadows-withal shadows-withal merged commit abd0ec3 into rust-lang:master Apr 12, 2020
alphamodel pushed a commit to wanglongqi/rustlings-solutions that referenced this pull request Apr 27, 2020
chore: update variables5.rs book link

chore: update variables5.rs book link
MendelMonteiro pushed a commit to MendelMonteiro/rustlings that referenced this pull request Jun 28, 2020
chore: update variables5.rs book link

chore: update variables5.rs book link
ppp3 pushed a commit to ppp3/rustlings that referenced this pull request May 23, 2022
chore: update variables5.rs book link

chore: update variables5.rs book link
dmoore04 pushed a commit to dmoore04/rustlings that referenced this pull request Sep 11, 2022
chore: update variables5.rs book link

chore: update variables5.rs book link
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