Skip to content

Feed cleanups#156019

Open
nnethercote wants to merge 2 commits intorust-lang:mainfrom
nnethercote:rm-Feed
Open

Feed cleanups#156019
nnethercote wants to merge 2 commits intorust-lang:mainfrom
nnethercote:rm-Feed

Conversation

@nnethercote
Copy link
Copy Markdown
Contributor

Two minor improvements. Details in the individual commits.

r? @oli-obk

RFC 430 says type parameters should be named "concise UpperCamelCase,
usually single uppercase letter". So either `Key` or `K` is more
appropriate than `KEY`, which looks like a constant.

I chose `K` because it's a well-known abbreviation of "key" used in lots
of places, e.g. `HashMap<K, V>`.
@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Apr 30, 2026
Copy link
Copy Markdown
Contributor

@oli-obk oli-obk left a comment

Choose a reason for hiding this comment

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

@rust-bors
Copy link
Copy Markdown
Contributor

rust-bors Bot commented May 1, 2026

📌 Commit ddc626d has been approved by oli-obk

It is now in the queue for this repository.

@rust-bors rust-bors Bot added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels May 1, 2026
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request May 1, 2026
Feed cleanups

Two minor improvements. Details in the individual commits.

r? @oli-obk
jhpratt added a commit to jhpratt/rust that referenced this pull request May 1, 2026
Feed cleanups

Two minor improvements. Details in the individual commits.

r? @oli-obk
rust-bors Bot pushed a commit that referenced this pull request May 1, 2026
Rollup of 6 pull requests

Successful merges:

 - #149637 (Do not run jump-threading for GPUs)
 - #155948 (Fix order-dependent visibility diagnostics)
 - #155600 (Adds a couple UI tests for polonius)
 - #155995 (-Zembed-source: also embed external source)
 - #156019 (Feed cleanups)
 - #156031 (Return a single diagnostic from `lex_token_trees`.)
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request May 1, 2026
Feed cleanups

Two minor improvements. Details in the individual commits.

r? @oli-obk
rust-bors Bot pushed a commit that referenced this pull request May 1, 2026
…uwer

Rollup of 9 pull requests

Successful merges:

 - #149637 (Do not run jump-threading for GPUs)
 - #154971 (Verify that penultimate segment of enum variant path refers to enum if it has args)
 - #155186 (Avoid loop_match self-assignment in MIR lowering)
 - #155948 (Fix order-dependent visibility diagnostics)
 - #156001 (ssa-range-prop: fix ICE when encountering self-domiating bb)
 - #155600 (Adds a couple UI tests for polonius)
 - #155995 (-Zembed-source: also embed external source)
 - #156019 (Feed cleanups)
 - #156031 (Return a single diagnostic from `lex_token_trees`.)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants