Uh oh!
There was an error while loading. Please reload this page.
Add help message for missing toolchain - #3939
Conversation
Uh oh!
There was an error while loading. Please reload this page.
rami3l
left a comment
There was a problem hiding this comment.
Thanks a lot for helping out! It'll be nice if you adjust our integration test accordingly as well :)
jtr860830
commented
Jul 11, 2024
OK, I'll adjust it. Thx for replying! |
@jtr860830 No worries! Just remember to squash everything into a single commit since it's a single unit of change: we use atomic commits across the repo. The integration test you need to fix is |
00ddad1 to
cf73e77Compare
rami3l
left a comment
There was a problem hiding this comment.
LGTM, and thanks again for helping out! :)
@jtr860830 Sorry, but I've changed my mind regarding this change, since it doesn't make the distinction between a typo (like Hint: Use |
jtr860830
commented
Jul 11, 2024
I get it. I'll improve it! |
cf73e77 to
c32a1e7CompareUh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
c32a1e7 to
6bcdf1fCompare6bcdf1f to
99d433cCompare
rami3l
left a comment
There was a problem hiding this comment.
Sorry for the delay, and thanks again for your help!
Co-authored-by: rami3l <rami3l@outlook.com>
99d433c to
28a3d79Comparejtr860830
commented
Jul 12, 2024
This PR has taught me a lot. Thx for your help! |
Closes#3573.
This patch adds a suggestion to run
rustup install <toolchain>for missing toolchain.