[WIP] Migrate rustc_ast_passes diagnostics to SessionDiagnostic and translatable messages (second part)#101657
[WIP] Migrate rustc_ast_passes diagnostics to SessionDiagnostic and translatable messages (second part)#101657finalchild wants to merge 23 commits intorust-lang:masterfrom
Conversation
|
r? @lcnr (rust-highfive has picked a reviewer for you, use r? to override) |
This comment has been minimized.
This comment has been minimized.
|
I don't understand how the CI job failed. Maybe some kind of incremental build issue? Is there a way to clean the cache of CI? |
I think ae49732 caused the error on your code. The commit has changed the |
5c81f5a to
1e828ee
Compare
|
r? @davidtwco |
|
Are you looking to land this or still working on it? |
|
There's one more file to migrate! |
This comment was marked as resolved.
This comment was marked as resolved.
f1d5454 to
5af4a77
Compare
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
06989c9 to
55a006f
Compare
This comment was marked as resolved.
This comment was marked as resolved.
55a006f to
ca14f39
Compare
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment was marked as resolved.
This comment was marked as resolved.
d6fa0da to
db90e4a
Compare
This comment was marked as resolved.
This comment was marked as resolved.
8035c7d to
3ca0f4b
Compare
davidtwco
left a comment
There was a problem hiding this comment.
Apologies for the delay in reviewing.
There was a problem hiding this comment.
Can't the derive be used for this?
There was a problem hiding this comment.
Can the derive be used for this?
There was a problem hiding this comment.
Can these be split into multiple Fluent messages to avoid pattern matching on strings?
|
☔ The latest upstream changes (presumably #104507) made this pull request unmergeable. Please resolve the merge conflicts. |
…mangleItemNonAscii
…athParam, ImplTraitTyNested, ImplTraitTyWithoutTraitBound, deprecated_extern_missing_abi
Co-authored-by: RanolP <public.ranolp@gmail.com>
Co-authored-by: RanolP <public.ranolp@gmail.com>
…cConstraintSuggestion
3ca0f4b to
8575873
Compare
This comment has been minimized.
This comment has been minimized.
|
☔ The latest upstream changes (presumably #104754) made this pull request unmergeable. Please resolve the merge conflicts. |
Second (and hopefully last) PR migrating rustc_ast_passes to SessionDiagnostics.
@rustbot label +A-translation