Uh oh!
There was an error while loading. Please reload this page.
Make --explain Handle Partial Error Codes - #28872
Conversation
rust-highfive
commented
Oct 6, 2015
Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @Aatch (or someone else) soon. If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. The way Github handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes. Please see the contribution instructions for more information. |
alexcrichton
commented
Oct 6, 2015
r? @Manishearth |
Manishearth
commented
Oct 6, 2015
@bors r+ Nice idea, thanks for the pull request! |
bors
commented
Oct 6, 2015
📌 Commit 185c3bb has been approved by |
bors
commented
Oct 6, 2015
⌛ Testing commit 185c3bb with merge e553542... |
bors
commented
Oct 7, 2015
💔 Test failed - auto-mac-64-opt |
Manishearth
commented
Oct 7, 2015
|
iwillspeak
commented
Oct 7, 2015
Should I squash the whitespace cleanup into the other commit, or do you want me to leave it as two separate commits? |
Manishearth
commented
Oct 7, 2015
Squash please |
Currently the explain command requires full erorr codes, complete with the leading zeros and the E at the beginning. This commit changes that, if you don't supply a full erorr code then the error number is padded out to the required size and the E is added to the beginning. This means that where previously you would need to write E0001, you can now write 0001, 001, 01 or jsut 1 to refer to the same error.
iwillspeak
commented
Oct 8, 2015
Ok. Squashed. |
Manishearth
commented
Oct 8, 2015
@bors r+ rollup |
bors
commented
Oct 8, 2015
📌 Commit 5d01556 has been approved by |
Currently the explain command line flag requires full error codes, complete with the leading zeros and the E at the beginning. This commit changes that, if you don't supply a full error code then the error number is padded out to the required size and the E is added to the beginning. This means that where previously you would need to write E0001, you can now write 0001, 001, 01 or just 1 to refer to the same error.
Currently the explain command line flag requires full error codes, complete with the leading zeros and the E at the beginning. This commit changes that, if you don't supply a full error code then the error number is padded out to the required size and the E is added to the beginning. This means that where previously you would need to write E0001, you can now write 0001, 001, 01 or just 1 to refer to the same error.
Currently the explain command line flag requires full error codes, complete with
the leading zeros and the E at the beginning. This commit changes that,
if you don't supply a full error code then the error number is padded
out to the required size and the E is added to the beginning.
This means that where previously you would need to write E0001, you can
now write 0001, 001, 01 or just 1 to refer to the same error.