-
Notifications
You must be signed in to change notification settings - Fork 531
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
Fix StringReader
references
#2230
Closed
nnethercote
wants to merge
2,506
commits into
rust-lang:master
from
nnethercote:fix-StringReader-refs
Closed
Fix StringReader
references
#2230
nnethercote
wants to merge
2,506
commits into
rust-lang:master
from
nnethercote:fix-StringReader-refs
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Most files names are relatively explicit and probably don't need to be explicited. However `rt.rs` is really not clear, and I believe it clarifies the text to indicate this is runtime service implementation.
* Add link in salsa * Update src/salsa.md Co-authored-by: León Orell Valerian Liehr <me@fmease.dev> * Update src/salsa.md Co-authored-by: León Orell Valerian Liehr <me@fmease.dev> --------- Co-authored-by: León Orell Valerian Liehr <me@fmease.dev>
Co-authored-by: Yuki Okushi <jtitor@2k36.org>
since this method has been renamed in rustc. Also remove a link to documentation in error-guaranteed.md because it was unused
Co-authored-by: Tshepang Mbambo <tshepang@gmail.com>
These were removed in 2021. rust-lang/rust#85993
* add implied bounds doc * lazy type aliases also have explicit implied bounds
* add opaque types doc * summary
* make shell.nix better * Mention using RUST_BOOTSTRAP_CONFIG * Move things to `buildInputs` and add `glibc.out glibc.static` This fixes the nofile-limit.rs UI test. * short lines for the short line fans * Fix pkgs
* Add some more details on feature gating * Apply suggestions from code review --------- Co-authored-by: Ross Smyth <rsmyth@electrocraft.com> Co-authored-by: Nilstrieb <48135649+Nilstrieb@users.noreply.github.com>
Co-authored-by: nora <48135649+Noratrieb@users.noreply.github.com>
But accept test names that include non-prefix issue number.
Co-authored-by: Jieyou Xu <39484203+jieyouxu@users.noreply.github.com> Co-authored-by: Tbkhi <me.stole546@silomails.com>
fix conflicting link names that lead to the "chapter link" leading to rustdoc's source code instead.
* Fix broken link in syntax-intro.md * Fix broken link in tests/compiletest.md * Fix incorrectly formatted reference-style link in compiler-src.md * Fix broken links to rustc-driver.md to point to rustc-driver/intro.md * Define URLs for incomplete links to rustc sources in stabilization_guide.md * Define link to coherence.md for incomplete link in solve/invariants.md
Co-authored-by: Jieyou Xu <jieyouxu@outlook.com>
`StringReader` was renamed `Lexer` in rust-lang/rust#133070. This commit updates the guide accordingly.
Whoa, git failure on my end. Let me fix this up. |
Ok, this was already fixed in #2208. |
🤏 close to getting right to the top spot of contributors to the dev guide by line number 😔 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
StringReader
was renamedLexer
in rust-lang/rust#133070. This commit updates the guide accordingly.