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

doc: Fix the 30-minute intro to use += operator. #16247

Merged
merged 1 commit into from
Aug 5, 2014

Conversation

TimDumol
Copy link
Contributor

@TimDumol TimDumol commented Aug 4, 2014

The 30-minute intro had a comment mentioning that += will be included in the future. It's already included, and this fixes it to use +=.

@alexcrichton
Copy link
Member

I think the comment still partially applies because the get_mut shouldn't be necessary, could you leave behind a part of the comment relevant to that aspect?

@TimDumol
Copy link
Contributor Author

TimDumol commented Aug 4, 2014

Will do!

The 30-minute intro had a comment mentioning that `+=` will be included in the future. It's already included, and this fixes it to use `+=`.
@TimDumol
Copy link
Contributor Author

TimDumol commented Aug 4, 2014

Done; made the comment a bit more specific too. Took some time because I had to clone the repo first.

bors added a commit that referenced this pull request Aug 5, 2014
The 30-minute intro had a comment mentioning that `+=` will be included in the future. It's already included, and this fixes it to use `+=`.
@bors bors closed this Aug 5, 2014
@bors bors merged commit a9979ad into rust-lang:master Aug 5, 2014
bors added a commit to rust-lang-ci/rust that referenced this pull request Feb 18, 2024
fix: Fix build scripts not being rebuilt in some occasions

Also makes proc-macro changed flag setting async, we don't wanna block `process_changes` on the database as that is on the main thread!
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