-
Notifications
You must be signed in to change notification settings - Fork 13.1k
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
migrate some rustc_borrowck
diagnostic
#100798
Conversation
Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @davidtwco (or someone else) soon. Please see the contribution instructions for more information. |
|
You don't need to invoke |
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 really great start! Thanks for the pull request, if you're intending to keep adding to this then we can mark it as a draft until you've added more, let me know.
|
||
borrowck_const_not_used_in_type_alias = | ||
const parameter `{$ct}` is part of concrete type but not \ |
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.
I'm not sure that \
is a linebreak in Fluent. You'll be able to check by running ./x.py test src/test/ui
and seeing if any of the tests fail.
use rustc_middle::ty::Ty; | ||
use rustc_span::Span; | ||
|
||
#[derive(SessionDiagnostic)] | ||
#[error(borrowck::move_borrowed, code = "E0505")] |
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 isn't used anywhere, right?
), | ||
) | ||
.emit(); | ||
//FIXME! |
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.
What needs fixed? :)
yes, please mark it as a draft. Sorry for the bot stuff, my intention was to wait in line quietly. |
The job Click to see the possible cause of the failure (guessed by this bot)
|
^ what the heck happened to this PR? cc @davidtwco Specifically, why did github think this got merged? |
wow, I'm terribly sorry.. Is there anything I should do? I think I was pushing to my own fork |
lol, you probably just need to open a new PR or something. |
@AndyJado can you (in the future) make separate branches for all things you want to add? This way github won't think that there is no commits, because you'll just use a separate branch and won't need to delete commits. |
That's alright, I think @WaffleLapkin's suggestion is correct, we've got some documentation on how we use Git that you might find helpful: https://rustc-dev-guide.rust-lang.org/git.html Just open a new PR with your changes :) |
@bors r+ rollup
it's my first pull request ever, I got all the time and energy, but I wonder if I'm heading the right place.🧎