Skip to content

Rename diagnostic::on_unmatch_args to diagnostic::on_unmatched_args#157887

Merged
rust-bors[bot] merged 2 commits into
rust-lang:mainfrom
fmease:rn-to-on-unmatched-args
Jun 14, 2026
Merged

Rename diagnostic::on_unmatch_args to diagnostic::on_unmatched_args#157887
rust-bors[bot] merged 2 commits into
rust-lang:mainfrom
fmease:rn-to-on-unmatched-args

Conversation

@fmease

@fmease fmease commented Jun 14, 2026

Copy link
Copy Markdown
Member

This renders the name of the attribute and the feature grammatical. #154794 (comment).

r? mejrs

@rustbot

rustbot commented Jun 14, 2026

Copy link
Copy Markdown
Collaborator

Some changes occurred to diagnostic attributes.

cc @mejrs

Some changes occurred in compiler/rustc_passes/src/check_attr.rs

cc @jdonszelmann, @JonathanBrouwer

Some changes occurred in compiler/rustc_attr_parsing

cc @jdonszelmann, @JonathanBrouwer

Some changes occurred in compiler/rustc_hir/src/attrs

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. T-libs Relevant to the library team, which will review and decide on the PR/issue. labels Jun 14, 2026
/// Allows deriving traits as per `SmartPointer` specification
(removed, derive_smart_pointer, "1.84.0", Some(123430), Some("replaced by `CoercePointee`"), 131284),
/// Allows macros to customize macro argument matcher diagnostics.
(removed, diagnostic_on_unmatch_args, "CURRENT_RUSTC_VERSION", Some(155642), Some("renamed to `diagnostic_on_unmatched_args`"), 157887),

@fmease fmease Jun 14, 2026

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

It's a bit unfortunate that we have to add a tombstone two months after the feature's introduction but oh well.

View changes since the review

@rust-log-analyzer

This comment has been minimized.

@fmease fmease force-pushed the rn-to-on-unmatched-args branch from f381afe to f1acff6 Compare June 14, 2026 13:10
@fmease fmease changed the title Rename diagnostics::on_unmatch_args to diagnostics::on_unmatched_args Rename diagnostic::on_unmatch_args to diagnostic::on_unmatched_args Jun 14, 2026
@fmease fmease force-pushed the rn-to-on-unmatched-args branch from f1acff6 to 69a17b1 Compare June 14, 2026 13:23

@mejrs mejrs 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.

Thanks! I do like this better. Also the attribute documentation efforts are much appreciated 🙂

@bors r+ rollup

View changes since this review

@rust-bors

rust-bors Bot commented Jun 14, 2026

Copy link
Copy Markdown
Contributor

📌 Commit 69a17b1 has been approved by mejrs

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 14, 2026
rust-bors Bot pushed a commit that referenced this pull request Jun 14, 2026
…uwer

Rollup of 5 pull requests

Successful merges:

 - #157677 (Slightly improve diagnostic impl+output for uncovered type parameters)
 - #157715 (Stop using a fn pointer to normalize_fn_sig inside TypeErrCtxt)
 - #157884 (Rename `errors.rs` file to `diagnostics.rs` (9/N))
 - #157887 (Rename `diagnostic::on_unmatch_args` to `diagnostic::on_unmatched_args`)
 - #157888 (Avoid `&raw` call recovery inside nested delimiters)
@rust-bors rust-bors Bot merged commit 0a4ee11 into rust-lang:main Jun 14, 2026
13 checks passed
@rustbot rustbot added this to the 1.98.0 milestone Jun 14, 2026
@fmease fmease deleted the rn-to-on-unmatched-args branch June 14, 2026 20:42
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. T-libs Relevant to the library team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants