Skip to content

Conversation

@scrabsha
Copy link
Contributor

@scrabsha scrabsha commented Nov 27, 2025

the title says everything. first commit adds a test that shows how current main behaves on misspelled item keyword. second commit adds the recovery, which allows to emit many more errors.

r? @jdonszelmann

@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 Nov 27, 2025
@rustbot
Copy link
Collaborator

rustbot commented Nov 29, 2025

rustc_errors::emitter was changed

cc @Muscraft

@scrabsha
Copy link
Contributor Author

@rustbot blocked on #149427.

@rustbot rustbot added S-blocked Status: Blocked on something else such as an RFC or other implementation work. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Nov 29, 2025
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Dec 1, 2025
…szelmann

Make the capitalization explicit on keyword misspell error

Will help for rust-lang#149405.
rust-timer added a commit that referenced this pull request Dec 1, 2025
Rollup merge of #149427 - scrabsha:push-rxkwyumxrrtu, r=jdonszelmann

Make the capitalization explicit on keyword misspell error

Will help for #149405.
@rustbot
Copy link
Collaborator

rustbot commented Dec 3, 2025

This PR was rebased onto a different main commit. Here's a range-diff highlighting what actually changed.

Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers.

@scrabsha
Copy link
Contributor Author

scrabsha commented Dec 3, 2025

ok, I should have tested before pushing. this is slightly broken and i need to figure out why

@rust-log-analyzer

This comment has been minimized.

@scrabsha
Copy link
Contributor Author

scrabsha commented Dec 3, 2025

ok, this is ready-ish. who could believe that parsing global static keywords in the wrong order would be an issue? 🤡

@rustbot ready

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-blocked Status: Blocked on something else such as an RFC or other implementation work. labels Dec 3, 2025
@jdonszelmann
Copy link
Contributor

@bors r+ rollup

@bors
Copy link
Collaborator

bors commented Dec 4, 2025

📌 Commit 9ffde14 has been approved by jdonszelmann

It is now in the queue for this repository.

@bors bors 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 Dec 4, 2025
bors added a commit that referenced this pull request Dec 4, 2025
Rollup of 9 pull requests

Successful merges:

 - #147224 (Emscripten: Turn wasm-eh on by default)
 - #149405 (Recover on misspelled item keyword)
 - #149443 (Tidying up UI tests [6/N])
 - #149524 (Move attribute safety checking to attribute parsing)
 - #149593 (powf, powi: point out SNaN non-determinism)
 - #149605 (Use branch name instead of HEAD when unshallowing)
 - #149612 (Apply the `bors` environment also to the `outcome` job)
 - #149623 (Don't require a normal tool build of clippy/rustfmt when running their test steps)
 - #149627 (Point to the item that is incorrectly annotated with `#[diagnostic::on_const]`)

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 55fa9f7 into rust-lang:main Dec 5, 2025
11 checks passed
@rustbot rustbot added this to the 1.93.0 milestone Dec 5, 2025
rust-timer added a commit that referenced this pull request Dec 5, 2025
Rollup merge of #149405 - scrabsha:push-tzonpluy, r=jdonszelmann

Recover on misspelled item keyword

the title says everything. first commit adds a test that shows how current `main` behaves on misspelled item keyword. second commit adds the recovery, which allows to emit many more errors.
github-actions bot pushed a commit to rust-lang/miri that referenced this pull request Dec 5, 2025
Rollup of 9 pull requests

Successful merges:

 - rust-lang/rust#147224 (Emscripten: Turn wasm-eh on by default)
 - rust-lang/rust#149405 (Recover on misspelled item keyword)
 - rust-lang/rust#149443 (Tidying up UI tests [6/N])
 - rust-lang/rust#149524 (Move attribute safety checking to attribute parsing)
 - rust-lang/rust#149593 (powf, powi: point out SNaN non-determinism)
 - rust-lang/rust#149605 (Use branch name instead of HEAD when unshallowing)
 - rust-lang/rust#149612 (Apply the `bors` environment also to the `outcome` job)
 - rust-lang/rust#149623 (Don't require a normal tool build of clippy/rustfmt when running their test steps)
 - rust-lang/rust#149627 (Point to the item that is incorrectly annotated with `#[diagnostic::on_const]`)

r? `@ghost`
`@rustbot` modify labels: rollup
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.

6 participants