Reconcile testfx vendored source baselines - #55372
Merged
Evangelink merged 2 commits intoJul 20, 2026
Merged
Conversation
Acknowledge the reviewed testfx terminal reporter and wire-contract drift without porting testfx-only structured progress or logging abstractions. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 7e20f449-2997-4207-9a29-265519e73fd3
|
Azure Pipelines: Successfully started running 1 pipeline(s). 2 pipeline(s) were filtered out due to trigger conditions. There may be pipelines that require an authorized user to comment /azp run to run. |
Contributor
There was a problem hiding this comment.
Pull request overview
This pull request updates the dotnet/sdk vendored-source manifest to reconcile reported drift against microsoft/testfx by advancing the tracked upstream baselines for the affected wire-contract constants and terminal reporter/terminal rendering files.
Changes:
- Bump
baseline_ref_sha/baseline_blob_shafor the eight driftedmicrosoft/testfxsources (wire constants + terminal reporter/terminal) ineng/vendored-files.json. - Align the manifest entries with the newly reviewed upstream blobs so
check_vendored_files.py validateno longer reports drift for those sources.
Show a summary per file
| File | Description |
|---|---|
| eng/vendored-files.json | Advances the recorded upstream baseline ref+blob SHAs for the drifted testfx wire contract and terminal reporter sources. |
Copilot's findings
- Files reviewed: 1/1 changed files
- Comments generated: 0
Evangelink
enabled auto-merge
July 20, 2026 11:12
3 tasks
Qualify the dotnet test exit-code constants after the shared test framework introduced a conflicting ExitCode type. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 7e20f449-2997-4207-9a29-265519e73fd3
0101
approved these changes
Jul 20, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
microsoft/testfxwire-contract and terminal-reporter driftmainrefDisplayMessage, and the rendering diagnostics rely on testfx-only logger/renderer abstractionsValidation
python .github/scripts/check_vendored_files.py validatemainCloses #55364
Closes #55365
Closes #55366
Closes #55367
Closes #55368
Closes #55369
Closes #55370
Closes #55371