Skip to content

Migrate more rustc_borrowck diagnostics to SessionDiagnostic - #100864

Merged
AndyJado merged 0 commit into
rust-lang:masterfrom
AndyJado:master
Aug 22, 2022
Merged

Migrate more rustc_borrowck diagnostics to SessionDiagnostic#100864
AndyJado merged 0 commit into
rust-lang:masterfrom
AndyJado:master

Conversation

@AndyJado

Copy link
Copy Markdown
Contributor

@rustbot label +A-translation

r? rust-lang/diagnostics

@rustbotrustbot added A-translation Area: Translation infrastructure, and migrating existing diagnostics to SessionDiagnostic T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Aug 22, 2022
@rust-highfive

Copy link
Copy Markdown
Contributor

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @TaKO8Ki (or someone else) soon.

Please see the contribution instructions for more information.

@rust-highfive

Copy link
Copy Markdown
Contributor

⚠️Warning⚠️

  • These commits modify submodules.

@rust-highfiverust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 22, 2022
@rust-log-analyzer

Copy link
Copy Markdown
Collaborator

The job mingw-check failed! Check out the build log: (web)(plain)

Click to see the possible cause of the failure (guessed by this bot)
 Checking rustc_codegen_ssa v0.0.0 (/checkout/compiler/rustc_codegen_ssa)
Checking rustc_mir_transform v0.0.0 (/checkout/compiler/rustc_mir_transform)
Checking rustc_borrowck v0.0.0 (/checkout/compiler/rustc_borrowck)
Checking rustc_codegen_llvm v0.0.0 (/checkout/compiler/rustc_codegen_llvm)
error: diagnostics should only be created in `SessionDiagnostic`/`AddSubdiagnostic` impls
|
|
857 | err.span_note(multi_span, "the used `impl` has a `'static` requirement");
|
note: the lint level is defined here
--> compiler/rustc_borrowck/src/diagnostics/region_errors.rs:2:9
|
|
2 | #![deny(rustc::diagnostic_outside_of_impl)]
error: diagnostics should be created using translatable messages
|
|
857 | err.span_note(multi_span, "the used `impl` has a `'static` requirement");
|
note: the lint level is defined here
--> compiler/rustc_borrowck/src/diagnostics/region_errors.rs:1:9
|
|
1 | #![deny(rustc::untranslatable_diagnostic)]
error: could not compile `rustc_borrowck` due to 2 previous errors
warning: build failed, waiting for other jobs to finish...
error: could not compile `rustc_borrowck` due to 2 previous errors

@bors

bors commented Aug 22, 2022

Copy link
Copy Markdown
Collaborator

☔ The latest upstream changes (presumably #100671) made this pull request unmergeable. Please resolve the merge conflicts.

@AndyJado
AndyJado merged commit d0ea1d7 into rust-lang:masterAug 22, 2022
@rustbotrustbot added this to the 1.65.0 milestone Aug 22, 2022
@davidtwcodavidtwco mentioned this pull request Aug 23, 2022
84 tasks
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-translationArea: Translation infrastructure, and migrating existing diagnostics to SessionDiagnosticS-waiting-on-reviewStatus: Awaiting review from the assignee but also interested parties.T-compilerRelevant 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

@AndyJado@rust-highfive@rust-log-analyzer@bors@TaKO8Ki@rustbot