-
Notifications
You must be signed in to change notification settings - Fork 12.7k
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
Bubble up an overflow error so that rustdoc can ignore it #55717
Conversation
@bors r+ |
📌 Commit 69c80f4 has been approved by |
(I personally vote for whack-a-mole, but what do I know...) |
@bors p=10 |
Bubble up an overflow error so that rustdoc can ignore it fixes #54524 Idk how to write a test for this, other than trying to minimize the entire diesel crate. If desirable I will do that. Note that there are many other such overflow errors hiding out there. Should we try to proactively eliminate them or do we just whack-a-mole them? cc @GuillaumeGomez r? @nikomatsakis
💔 Test failed - status-travis |
Your PR failed on Travis (raw log). Through arcane magic we have determined that the following fragments from the build log may contain information about the problem. Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
@bors retry got stuck during initial checkout |
⌛ Testing commit 69c80f4 with merge da812b05151db5bbf3e69f455ebdcfdf9e203a51... |
💔 Test failed - status-travis |
⌛ Testing commit 69c80f4 with merge 1a1374bb6d7e3ce453d9e216da3b16ac35c6f201... |
💔 Test failed - status-travis |
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
@bors retry 9696 |
⌛ Testing commit 69c80f4 with merge 1c953cdf998080d5457b3453ad1f0ee5636539b4... |
💔 Test failed - status-appveyor |
@rust-timer build 1c953cdf998080d5457b3453ad1f0ee5636539b4 I find it strange this keeps timing out. @bors retry |
Success: Queued 1c953cdf998080d5457b3453ad1f0ee5636539b4 with parent 36a50c2, comparison URL. |
Finished benchmarking try commit 1c953cdf998080d5457b3453ad1f0ee5636539b4 |
Bubble up an overflow error so that rustdoc can ignore it fixes #54524 Idk how to write a test for this, other than trying to minimize the entire diesel crate. If desirable I will do that. Note that there are many other such overflow errors hiding out there. Should we try to proactively eliminate them or do we just whack-a-mole them? cc @GuillaumeGomez r? @nikomatsakis
☀️ Test successful - status-appveyor, status-travis |
fixes #54524
Idk how to write a test for this, other than trying to minimize the entire diesel crate. If desirable I will do that.
Note that there are many other such overflow errors hiding out there. Should we try to proactively eliminate them or do we just whack-a-mole them?
cc @GuillaumeGomez
r? @nikomatsakis