-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Doctests: Enable running doc tests for complexity lints #4325
Conversation
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.
Thanks! r=me after NIT fix.
dc43389
to
abfa8a9
Compare
@bors r=flip1995 |
📌 Commit abfa8a9 has been approved by |
Doctests: Enable running doc tests for complexity lints changelog: none master: `113 passed; 0 failed; 91 ignored; 0 measured; 0 filtered out` this PR: `181 passed; 0 failed; 110 ignored; 0 measured; 0 filtered out` cc #4319
💔 Test failed - status-appveyor |
This should be possible to merge independently of rust-lang#4325 cc rust-lang#4319
Because there were fixed in the parent commit.
@bors r=flip1995 |
📌 Commit eb68dc9 has been approved by |
Doctests: Enable running doc tests for complexity lints changelog: none master: `113 passed; 0 failed; 91 ignored; 0 measured; 0 filtered out` this PR: `181 passed; 0 failed; 110 ignored; 0 measured; 0 filtered out` cc #4319
💔 Test failed - checks-travis |
@bors retry (spurious) |
Doctests: Enable running doc tests for complexity lints changelog: none master: `113 passed; 0 failed; 91 ignored; 0 measured; 0 filtered out` this PR: `181 passed; 0 failed; 110 ignored; 0 measured; 0 filtered out` cc #4319
☀️ Test successful - checks-travis, status-appveyor |
changelog: none
master:
113 passed; 0 failed; 91 ignored; 0 measured; 0 filtered out
this PR:
181 passed; 0 failed; 110 ignored; 0 measured; 0 filtered out
cc #4319