improve error message when a cfg-ed out item is resolved#154037
improve error message when a cfg-ed out item is resolved#154037scrabsha wants to merge 1 commit intorust-lang:mainfrom
Conversation
|
Failed to set assignee to
|
|
Failed to set assignee to
|
|
@rustbot reroll |
r? estebank |
|
@Kivooeo if you don't mind me asking, why did it fail when i opened the PR? i remember r?ing people in the past, i'm a bit confused :/ edit: oah i did not suspect this would trigger rustbot again, sorry for the noise edit: i think this is because rustbot looks for the first occurrence of r question mark anywhere in the message and tries to use that (i thought it had to be at the beginning of a line). |
|
Failed to set assignee to
|
|
Might be good to also special case |
d07ece2 to
995f854
Compare
|
done, thanks! |
The issue is that the r-? command assigns the reviewer based on the word that follows it. In your first attempt, you wrote "r-? you Esteban" -- but "you" isn't a valid reviewer. Then you tried "estebank~~", which is also incorrect To assign a reviewer correctly, you need to use their exact GitHub username. In this case, it should be "estebank" |
the current note for "cannot find function
fin this scope" is:i think mentioning only
barin the label is misleading. in more complicated setups, this leads to obscure notes such as "the item is gated behind theptrfeature".this PR improves the situation by naming both the name and value in the label (in the example i linked,
target_has_atomicandptr). while i was at it, i also special-cased#[cfg(false)],#[cfg(any())]and#[cfg(feature = "meow")](aka Cargo features).i also tried to use
#[derive(SubDiagnostic)]onFoundItemConfigureOut, with no success (the resulting diagnostic was showing multiple help snippets, i couldn't figure out how to merge them). any help would be appreciated.r? you Esteban, as you added the diagnostic initially - feel free to unassign yourself if necessary :)r? @estebanknevermind apparently that's not possible --'