Skip to content

Tutorial #13907

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

Merged
merged 3 commits into from
May 5, 2014
Merged

Tutorial #13907

merged 3 commits into from
May 5, 2014

Conversation

iliekturtles
Copy link
Contributor

Some minor clarifications and fixes.

@iliekturtles
Copy link
Contributor Author

I thought I compiled locally after making the third commit. Is there a specific make target for documentation tests? I rebased and am recompiling locally again to attempt to reproduce the error.

@huonw
Copy link
Member

huonw commented May 5, 2014

The tutorial can be checked via make check-stage2-doc-tutorial.

Combined redundant paragraphs about the match expression and removed a
redundant example.
Corrected example to to use Rand trait referenced in preceding
description and included an example using the Show trait to print ABC.
@iliekturtles
Copy link
Contributor Author

@huonw, thanks! That did the trick. Corrections have been pushed and are ready for review.

bors added a commit that referenced this pull request May 5, 2014
@bors bors closed this May 5, 2014
@bors bors merged commit 055cbde into rust-lang:master May 5, 2014
@iliekturtles iliekturtles deleted the tutorial branch May 8, 2014 15:59
bors added a commit to rust-lang-ci/rust that referenced this pull request Feb 13, 2023
Don't include lifetime or label apostrophe when renaming

Closes rust-lang#13907
flip1995 pushed a commit to flip1995/rust that referenced this pull request Jan 9, 2025
Using `lifetime.ident.name` in suggestions will not output the raw
modifier. For example, `'r#struct` will be rendered as `'struct` which
would be incorrect.

Fix rust-lang#13899

changelog: [`needless_arbitrary_self_type`]: use the raw lifetime name
in suggestions
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.

4 participants