-
Notifications
You must be signed in to change notification settings - Fork 406
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
Increase CI coverage #612
Increase CI coverage #612
Conversation
CC @UebelAndre |
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.
Sounds good. I'm curious how you actually ensure you're trying to enable more. If there's an automated way to remind you you're not testing everything or try to enable tests and see if something passes based on some change.
I don't think they will be fixed accidentally, so a system automatically checking if they pass won't be very useful. I'll create issues for them and we should try to fix them soon. |
4eae91b
to
9c5828a
Compare
I realized we don't run tests on Windows. This PR fixes that. While there I also enabled tests on RBE.