-
Notifications
You must be signed in to change notification settings - Fork 12.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Porting 'compiler/rustc_trait_selection' to translatable diagnostics - Part 1 #100814
Porting 'compiler/rustc_trait_selection' to translatable diagnostics - Part 1 #100814
Conversation
Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @compiler-errors (or someone else) soon. Please see the contribution instructions for more information. |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a great start, left a comment or two
compiler/rustc_error_messages/locales/en-US/trait_selection.ftl
Outdated
Show resolved
Hide resolved
r? @davidtwco |
This comment was marked as resolved.
This comment was marked as resolved.
We have a no merge policy, do you think you could remove those merge commits? :) |
2fbc967
to
1b81e3a
Compare
rustc_trait_selection
diagnostics
cc @davidtwco, @compiler-errors, @JohnTitor, @estebank, @TaKO8Ki |
1b81e3a
to
18feed3
Compare
1d24c51
to
013994a
Compare
413d176
to
06a2d5b
Compare
This comment was marked as resolved.
This comment was marked as resolved.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
r=me after rebase
06a2d5b
to
8e82200
Compare
@bors r+ |
…diagnostics, r=davidtwco Porting 'compiler/rustc_trait_selection' to translatable diagnostics - Part 1 `@rustbot` label +A-translation r? rust-lang/diagnostics cc rust-lang#100717
…iaskrgr Rollup of 6 pull requests Successful merges: - rust-lang#100121 (Try normalizing types without RevealAll in ParamEnv in MIR validation) - rust-lang#100200 (Change implementation of `-Z gcc-ld` and `lld-wrapper` again) - rust-lang#100814 ( Porting 'compiler/rustc_trait_selection' to translatable diagnostics - Part 1) - rust-lang#101215 (Also replace the version placeholder in rustc_attr) - rust-lang#101260 (Use `FILE_ATTRIBUTE_TAG_INFO` to get reparse tag) - rust-lang#101323 (Remove unused .toggle-label CSS rule) Failed merges: r? `@ghost` `@rustbot` modify labels: rollup
@rustbot label +A-translation
r? rust-lang/diagnostics
cc #100717