-
Notifications
You must be signed in to change notification settings - Fork 330
Remove the bazel test runner option #3371
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
Remove the bazel test runner option #3371
Conversation
} | ||
}); | ||
// Only enable scope changes if the new bazel test runner is enabled. | ||
// If the new runner is disabled, all scopes will be blaze target-level. |
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.
We can update this comment.
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.
Done.
ping re: landing this |
I started a thread on the g3 users announcement list for this. Policy is to allow 1 week for discussion before landing the potentially breaking change. |
We are starting the next build. If we want this is this month's release, we'll need to land this. |
No objections internally, landing it. |
* Remove the bazel test runner option * Respond to Devon's comments
Remove it and clear out code paths that involve the user preference.