Skip to content

[tiny] Use multipart suggestion for finding ident when literal was expected in attr#157514

Merged
rust-bors[bot] merged 1 commit into
rust-lang:mainfrom
estebank:attr-expected-lit
Jul 18, 2026
Merged

[tiny] Use multipart suggestion for finding ident when literal was expected in attr#157514
rust-bors[bot] merged 1 commit into
rust-lang:mainfrom
estebank:attr-expected-lit

Conversation

@estebank

@estebank estebank commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

Minor rendering tweak.

@rustbot

rustbot commented Jun 5, 2026

Copy link
Copy Markdown
Collaborator

Some changes occurred in compiler/rustc_attr_parsing

cc @jdonszelmann, @JonathanBrouwer

@rustbot rustbot added A-attributes Area: Attributes (`#[…]`, `#![…]`) 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 Jun 5, 2026
@rustbot

rustbot commented Jun 5, 2026

Copy link
Copy Markdown
Collaborator

r? @mejrs

rustbot has assigned @mejrs.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: compiler
  • compiler expanded to 73 candidates
  • Random selection from 20 candidates

Comment thread compiler/rustc_attr_parsing/src/parser.rs Outdated

@JonathanBrouwer JonathanBrouwer left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

r=me unless you want to do something with Kivooeo's suggestion

View changes since this review

@JonathanBrouwer JonathanBrouwer left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@rust-bors

rust-bors Bot commented Jun 13, 2026

Copy link
Copy Markdown
Contributor

📌 Commit 1ade5a2 has been approved by JonathanBrouwer

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 Jun 13, 2026
jhpratt added a commit to jhpratt/rust that referenced this pull request Jun 13, 2026
…thanBrouwer

[tiny] Use multipart suggestion for finding ident when literal was expected in attr

Minor rendering tweak.
rust-bors Bot pushed a commit that referenced this pull request Jun 13, 2026
Rollup of 9 pull requests

Successful merges:

 - #157653 (remove AliasTerm::def_id())
 - #157773 (Remove AnonConstKind::GCA and reject generic anon consts)
 - #157846 (Avoid ICE in unconstrained generic parameter suggestion for enums)
 - #157514 ([tiny] Use multipart suggestion for finding ident when literal was expected in attr)
 - #157740 (rustdoc: correct doctest span for trailing semicolon after item)
 - #157782 (Added `PhantomPinned` diagnostic item and prevented dead field warning on `PhantomPinned`)
 - #157802 (Ensure that optimize attributes on closures are inherited by the shim.)
 - #157812 (rustc_public: implement `ty::Alias` conversion)
 - #157833 (Update wasm-component-ld to 0.5.25)
@samueltardieu

Copy link
Copy Markdown
Member

@bors r-
Failed in CI

@rust-bors rust-bors Bot added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Jun 13, 2026
@rust-bors

rust-bors Bot commented Jun 13, 2026

Copy link
Copy Markdown
Contributor

This pull request was unapproved.

This PR was contained in a rollup (#157851), which was unapproved.

View changes since this unapproval

@estebank
estebank force-pushed the attr-expected-lit branch from 1ade5a2 to 6bf3fbe Compare July 15, 2026 17:12
@JonathanBrouwer

Copy link
Copy Markdown
Contributor

@bors try jobs=aarch64-apple

@rust-bors

This comment has been minimized.

rust-bors Bot pushed a commit that referenced this pull request Jul 17, 2026
[tiny] Use multipart suggestion for finding ident when literal was expected in attr


try-job: aarch64-apple
@rust-bors

rust-bors Bot commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

☀️ Try build successful (CI)
Build commit: 8816969 (88169693f8cf757cae5b0dc434877b358a2abfa9)
Base parent: 4a9d536 (4a9d5368df6450bbd2fc8dde4508c3e5d83bb19d)

@JonathanBrouwer

Copy link
Copy Markdown
Contributor

@bors r+ rollup

@rust-bors

rust-bors Bot commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

📌 Commit 6bf3fbe has been approved by JonathanBrouwer

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-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Jul 17, 2026
rust-bors Bot pushed a commit that referenced this pull request Jul 17, 2026
…uwer

Rollup of 5 pull requests

Successful merges:

 - #158544 (Move `std::io::Read` to `alloc::io`)
 - #158668 (`DeeplyNormalize` and `normalize_with_depth_to` take `Unnormalized<T>` as input)
 - #157514 ([tiny] Use multipart suggestion for finding ident when literal was expected in attr)
 - #158748 (improve diagnostic suggestion when matching struct variants with tuple)
 - #159463 (triagebot: tweak wording for miri changes message)

Failed merges:

 - #158916 (Emit all borrowck errors at the end of borrowck)
@rust-bors
rust-bors Bot merged commit cadbb17 into rust-lang:main Jul 18, 2026
14 checks passed
@rustbot rustbot added this to the 1.99.0 milestone Jul 18, 2026
rust-timer added a commit that referenced this pull request Jul 18, 2026
Rollup merge of #157514 - estebank:attr-expected-lit, r=JonathanBrouwer

[tiny] Use multipart suggestion for finding ident when literal was expected in attr

Minor rendering tweak.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-attributes Area: Attributes (`#[…]`, `#![…]`) 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.

6 participants