Skip to content

Fix more test greps in invalid Unicode#28852

Open
fmeum wants to merge 1 commit intobazelbuild:masterfrom
fmeum:fix-expect-log
Open

Fix more test greps in invalid Unicode#28852
fmeum wants to merge 1 commit intobazelbuild:masterfrom
fmeum:fix-expect-log

Conversation

@fmeum
Copy link
Collaborator

@fmeum fmeum commented Mar 2, 2026

Description

Motivation

Fixes #28726

Build API Changes

No

Checklist

  • I have added tests for the new use cases (if any).
  • I have updated the documentation (if applicable).

Release Notes

RELNOTES: None

Copy link
Member

@meteorcloudy meteorcloudy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

We should be able to verify with

export USE_BAZEL_VERSION=last_green
bazel test //src/test/shell/integration:bazel_query_test

@meteorcloudy meteorcloudy added awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally and removed awaiting-review PR is awaiting review from an assigned reviewer labels Mar 3, 2026
@fmeum
Copy link
Collaborator Author

fmeum commented Mar 3, 2026

I ran that and it passed, but for whatever that's worth I also did that for the last PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bazel CI] Integration Failure: Multi-shard Regression in bazel_query_test on macOS (altered locale environment is causing problems on the macOS CI)

2 participants