Skip to content

Minor tweaks to code and tests - #223

Merged
todi1856 merged 5 commits into
masterfrom
minor-tweaks
Sep 23, 2026
Merged

todi1856 merged 5 commits into
masterfrom
minor-tweaks

Conversation

@todi1856

@todi1856 todi1856 commented Sep 23, 2026 •

Copy link
Copy Markdown
Member

Previously logcat Integration tests were running on NVIDIA Shield, switched them to Pixel Pro 8.

Also we now reports logcat/screenshot artifacts for integration tests.

Also cherry picked few changes from live stream PR, to make upcoming PR smaller.

Other than that, there's no changes to logcat

@todi1856

Copy link
Copy Markdown
Member Author

/review

@u-pr

u-pr Bot commented Sep 23, 2026

Copy link
Copy Markdown
Contributor

ℹ️ Skipped — a harness review for this commit (73dc549) already ran or is in flight. Same-commit re-runs are paused for 10 min; push a new commit, or re-comment /harness-review in ~10 min to force a fresh one.

🤖 Helpful? 👍/👎

@u-pr u-pr Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

💡 Harness Review

Ship it

The changes preserve capture behavior while improving device setup, diagnostics, and packed-package CI coverage; the associated ADB, lifecycle, artifact, reflection, and job paths showed no actionable defects.

Reviewed commit 73dc549

🤖 Helpful? 👍/👎

@todi1856
todi1856 requested a review from manugildev September 23, 2026 07:58
Comment thread .yamato/upm-integration-ci.yml Outdated
Co-authored-by: Manuel Gil <manugildev@gmail.com>
@todi1856
todi1856 merged commit 3923da6 into master Sep 23, 2026
17 of 22 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.

2 participants