Skip to content

Fix demo utils test + clean-ups after PR#7120#7121

Merged
samanoelton merged 3 commits into
tensorflow:masterfrom
psamanoelton:fix_demo_utils_test
Jun 12, 2026
Merged

Fix demo utils test + clean-ups after PR#7120#7121
samanoelton merged 3 commits into
tensorflow:masterfrom
psamanoelton:fix_demo_utils_test

Conversation

@psamanoelton

@psamanoelton psamanoelton commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

Move TF dependency to deps in demo_utils_test to fix a linkage issue.

1st and 2nd commit to fix Buildifier internal checks.

Comment thread third_party/repo.bzl

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

These changes are not mentioned in the description. Are they intended to be merged together? If so, please update the description, else revert these files?

@arcra

arcra commented Jun 12, 2026

Copy link
Copy Markdown
Member

I think there's a missing space in the description, too.

@psamanoelton psamanoelton changed the title Fix demo utils test Fix demo utils test + clean-ups after PR#7120 Jun 12, 2026
@psamanoelton

Copy link
Copy Markdown
Contributor Author

This PR contains also the clean-ups of #7121

@samanoelton samanoelton merged commit 0675a15 into tensorflow:master Jun 12, 2026
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants