-
Notifications
You must be signed in to change notification settings - Fork 234
Fix serial doctest + require passing tests on bors #305
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
Conversation
r? @therealprof (rust-highfive has picked a reviewer for you, use r? to override) |
Ah, #300 let to the example actually being build, which contains compiler errors. Sorry for the inconvenience 🙈 😀 |
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.
LGTM, thanks.
bors r+
305: Fix serial doctest + require passing tests on bors r=therealprof a=eldruin This escaped CI somehow in #300. Thanks to `@adamgreig` for his help in figuring this out. Now passing the tests is also a requirement for bors. Co-authored-by: Diego Barrios Romero <[email protected]>
Timed out. |
bors retry |
305: Fix serial doctest + require passing tests on bors r=therealprof a=eldruin This escaped CI somehow in #300. Thanks to `@adamgreig` for his help in figuring this out. Now passing the tests is also a requirement for bors. Co-authored-by: Diego Barrios Romero <[email protected]>
Co-authored-by: Adam Greig <[email protected]>
Canceled. |
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.
Let's try this then. ;)
bors r+
This escaped CI somehow in #300.
Thanks to @adamgreig for his help in figuring this out.
Now passing the tests is also a requirement for bors.