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

update error code docs to reflect recent changes #1625

Merged
merged 2 commits into from
Mar 3, 2023

Conversation

Ezrashaw
Copy link
Contributor

Clean up the error code sections to reflect recent changes in the space.

See also: rust-lang/rust#108482

cc @GuillaumeGomez

@JohnTitor
Copy link
Member

Could you add a redirect config below not to break the current URL?

[output.html.redirect]

@Ezrashaw
Copy link
Contributor Author

Ezrashaw commented Feb 27, 2023

@JohnTitor Fixed, the other diagnostics redirect wasn't working as well (including on the real website). Obviously I'm new to rustc-dev-guide so there might be other things I've missed.

src/diagnostics.md Outdated Show resolved Hide resolved
@Ezrashaw
Copy link
Contributor Author

@GuillaumeGomez All done, thanks for the feedback.

@GuillaumeGomez
Copy link
Member

Thanks!

@bors r=JohnTitor,GuillaumeGomez rollup

@Ezrashaw
Copy link
Contributor Author

Ezrashaw commented Mar 1, 2023

@GuillaumeGomez I don't think this repo uses bors? PRs are just merged manually? Not entirely sure, never contributed to this repo.

@GuillaumeGomez
Copy link
Member

Sorry, force of habit. Let me ping someone.

cc @matthiaskrgr

@matthiaskrgr
Copy link
Member

Huh? I've never really interacted with this repo, sorry 😄

@GuillaumeGomez
Copy link
Member

I saw your name in other PRs so I summoned you. My bad. 😆

cc @tshepang

Copy link
Member

@JohnTitor JohnTitor left a comment

Choose a reason for hiding this comment

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

Looks great, thanks!

@JohnTitor JohnTitor merged commit 2d0681e into rust-lang:master Mar 3, 2023
@Ezrashaw Ezrashaw deleted the update-error-explanations branch March 3, 2023 21:48
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Mar 13, 2023
Update books

## rust-lang/nomicon

1 commits in 79b53665a7c61d171fb8c5ad0b73b371f9ee6ba7..1f3e4cd4fd88b5b5d45feb86a11b6d2f93e5a974
2023-03-12 21:51:29 UTC to 2023-03-12 21:51:29 UTC

- Change incorrect reference to "above" in ffi.md (rust-lang/nomicon#404)

## rust-lang/reference

4 commits in a9afb04b47a84a6753e4dc657348c324c876102c..24c87f6663aed55b05d2cc286878f28f21918825
2023-03-11 13:53:24 UTC to 2023-02-28 16:07:20 UTC

- Document movbe target feature (rust-lang/reference#1336)
- fix typo (rust-lang/reference#1339)
- fix: Typo/reference (rust-lang/reference#1338)
- Document `cmpxchg16b` target feature (rust-lang/reference#1331)

## rust-lang/rustc-dev-guide

13 commits in b06dab84083390e0ee1e998f466545a8a1a76a9f..b1b6d693cd1461e53de4132c1b183ace31cd36e5
2023-03-13 02:42:00 UTC to 2023-03-03 11:12:51 UTC

- Rename `config.toml.example` to `config.example.toml` (rust-lang/rustc-dev-guide#1641)
- fix typo `SubstRef` to `SubstsRef` (rust-lang/rustc-dev-guide#1642)
- tracing log along diff crates (rust-lang/rustc-dev-guide#1640)
- Fix Rust Analyzer settings location (rust-lang/rustc-dev-guide#1637)
- Reflect the changes in rust-lang#67000 (rust-lang/rustc-dev-guide#1632)
- improve capacity note (rust-lang/rustc-dev-guide#1634)
- Remove mention to lexer/parser refactoring (rust-lang/rustc-dev-guide#1629)
- Update date reference about infer context variables (rust-lang/rustc-dev-guide#1630)
- Update explnation about `Body.basic_blocks` (rust-lang/rustc-dev-guide#1631)
- Add detail to contributing guide (rust-lang/rustc-dev-guide#1628)
- Making the sentence more clear (rust-lang/rustc-dev-guide#1624)
- Fixed typing error (rust-lang/rustc-dev-guide#1623)
- update error code docs to reflect recent changes (rust-lang/rustc-dev-guide#1625)
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