Skip to content

Rustup #10802

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 21 commits into from
May 20, 2023
Merged

Rustup #10802

merged 21 commits into from
May 20, 2023

Conversation

flip1995
Copy link
Member

r? @ghost

changelog: none

jyn514 and others added 19 commits May 1, 2023 21:44
This greatly simplifies how hard it is to set a custom bug report url; previously tools had to copy
the entire hook implementation.

- Switch clippy to the new hook

  This also adds a `extra_info` callback so clippy can include its own version number, which differs
  from rustc's.

- Call `install_ice_hook` in rustfmt
Implement RFC 3348, `c"foo"` literals

RFC: rust-lang/rfcs#3348
Tracking issue: #105723
Make the BUG_REPORT_URL configurable by tools

This greatly simplifies how hard it is to set a custom bug report url; previously tools had to copy
the entire hook implementation.

I haven't changed clippy in case they want to make the change upstream instead of the subtree, but
I'm happy to do so here if the maintainers want - cc ````@rust-lang/clippy````

Fixes rust-lang/rust#109486.
…errors

Introduce `AliasKind::Inherent` for inherent associated types

Allows us to check (possibly generic) inherent associated types for well-formedness.
Type inference now also works properly.

Follow-up to #105961. Supersedes #108430.
Fixes #106722.
Fixes #108957.
Fixes #109768.
Fixes #109789.
Fixes #109790.

~Not to be merged before #108860 (`AliasKind::Weak`).~

CC `@jackh726`
r? `@compiler-errors`

`@rustbot` label T-types F-inherent_associated_types
It's just a short wrapper used by `tcx.require_lang_item`. Deleting it
gives us a negative diff.
Remove `LangItems::require`

It's just a short wrapper used by `tcx.require_lang_item`. Deleting it gives us a negative diff.
@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label May 20, 2023
@flip1995
Copy link
Member Author

@bors r+

@bors
Copy link
Contributor

bors commented May 20, 2023

📌 Commit 6c4097c has been approved by flip1995

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented May 20, 2023

⌛ Testing commit 6c4097c with merge 1aee538...

bors added a commit that referenced this pull request May 20, 2023
Rustup

r? `@ghost`

changelog: none
@bors
Copy link
Contributor

bors commented May 20, 2023

💔 Test failed - checks-action_test

@flip1995
Copy link
Member Author

@bors r+

@bors
Copy link
Contributor

bors commented May 20, 2023

📌 Commit 2e95a4f has been approved by flip1995

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented May 20, 2023

⌛ Testing commit 2e95a4f with merge 435a8ad...

@bors
Copy link
Contributor

bors commented May 20, 2023

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: flip1995
Pushing 435a8ad to master...

@bors bors merged commit 435a8ad into rust-lang:master May 20, 2023
@flip1995 flip1995 deleted the rustup branch May 20, 2023 13:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties
Projects
None yet
Development

Successfully merging this pull request may close these issues.