-
Notifications
You must be signed in to change notification settings - Fork 12.8k
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
clean up E0718 explanation #74123
clean up E0718 explanation #74123
Conversation
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.
Looks good to me.
@bors: r=pickfire rollup |
📌 Commit 4728438 has been approved by |
🌲 The tree is currently closed for pull requests below priority 30, this pull request will be tested once the tree is reopened |
@GuillaumeGomez Can I even do rollups? I thought it needs to be authorized users? |
@pickfire this just marks it safe for a rollup, you don't have to do the rollup :D |
…kfire clean up E0718 explanation r? @Dylan-DPC
…kfire clean up E0718 explanation r? @Dylan-DPC
…kfire clean up E0718 explanation r? @Dylan-DPC
…kfire clean up E0718 explanation r? @Dylan-DPC
…kfire clean up E0718 explanation r? @Dylan-DPC
…kfire clean up E0718 explanation r? @Dylan-DPC
…arth Rollup of 8 pull requests Successful merges: - rust-lang#73354 (Update RELEASES.md for 1.45.0) - rust-lang#73852 (rustdoc: insert newlines between attributes) - rust-lang#73867 (Document the union keyword) - rust-lang#74046 (Fix caching issue when building tools.) - rust-lang#74123 (clean up E0718 explanation) - rust-lang#74147 (rustdoc: Allow linking from private items to private types) - rust-lang#74285 (rust-lang#71669: add ui, codegen tests for volatile + nearby int intrinsics) - rust-lang#74286 (Added detailed error code explanation for issue E0688 in Rust compiler.) Failed merges: r? @ghost
r? @Dylan-DPC