-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
Resume validator tests by using bazel 5 #38688
Conversation
@estherkim |
Done! |
Warning: disparity between this PR Percy build and its The Percy build for this PR was approved (either manually by a member of the AMP team, or automatically if there were no visual diffs). However, during a continuous integration step we generated another Percy build using the commit on the This is possibly an indication of an issue with this pull request, but could also be the result of flakiness. Please inspect the two builds < This PR's Percy build /
|
Hi @antiphoton - looks like validator tests fail on main. Is Bazel installed correctly? |
another ping @antiphoton, @amaltas - a failure on main is blocking nightly cuts, which will prevent a new beta for next week's release cycle. This possibly wasn't working before merging. I don't see |
This reverts commit 13cfe7f.
Fix #38610