Test Snapshot Completion Automation - #7
Open
nagilson wants to merge 3 commits into
Open
Conversation
nagilson
changed the base branch from
release/10.0.1xx
to
nagilson-clean-merge-jjonescz-sprint-cln
December 10, 2025 19:37
Owner
Author
|
/fixcompletions |
1 similar comment
Owner
Author
|
/fixcompletions |
|
❌ Failed to update completion snapshots. Please check the workflow logs for details. |
Owner
Author
|
/fixcompletions |
|
ℹ️ No completion snapshot files needed to be updated. |
Owner
Author
|
/fixcompletions |
|
|
|
ℹ️ No completion snapshot files needed to be updated. Review the workflow run. |
Owner
Author
|
/fixcompletions |
1 similar comment
Owner
Author
|
/fixcompletions |
|
|
|
✅ CLI completion snapshots have been updated and committed to this PR. See workflow details. |
nagilson
added a commit
to dotnet/sdk
that referenced
this pull request
Dec 12, 2025
Fixes bugs from copilot that makes the tasks work. For evidence of this working and a demo, please see my fork which allowed me to test the actions by pushing to my own main, without running in prod. The most recent comments as of now which reflect the current code working are nagilson#4 (comment) and nagilson#7 (comment) nagilson#4 Adds /completions and /xlf as shorter aliases. Also remarks on /backport in the dev guide - it's not directly in the SDK but we use it a lot. It might be worth porting to arcade for the xlf one :)
This reverts commit f746483.
nagilson
pushed a commit
that referenced
this pull request
Mar 13, 2026
Changes include: - Rename 'managed installations' to 'tracked installations' (#10) - Fix '.metadata' -> 'metadata' folder name (#18) - Include component type in error/warning messages (#11) - Remove component spec from runtime update (#12) - Remove GetInstallations wrapper (#6) - ScopedMutex throws TimeoutException on failure (#22) - Make InstallResult.Install non-nullable (#17) - Remove ComponentFilesExist shortcut (#19) - Wire up InstallSource enum (#21) - ListCommand: enum types, narrowed mutex, read-only verify, spacing (#3,#4,#5,#1) - Show global.json path in list output (#15) - Add doc comments to ListData types (#7) - Move SourceOption to CommonOptions, remove alias fields (#9) - Remove ResolveChannelFromGlobalJson test hook wrapper (#14) - Root 'dotnetup update' updates all components (#16) - GlobalJsonChannelResolver: support rollForward policy (#13) - Error when installing to unmanaged dotnet root (#20) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
nagilson
pushed a commit
that referenced
this pull request
Mar 13, 2026
Changes include: - UpdateWorkflow.Execute returns non-zero exit code on failure (#1) - InfoCommand JSON output includes InstallSpecs (#2) - Wire --update-global-json through SdkUpdateCommand to UpdateWorkflow (#3) - Implement DotnetInstallManager.UpdateGlobalJson using Utf8JsonReader (#3) - Fix ResolveEntryDestPath to use normalized entry name (#4) - Fix test: MajorMinor test uses correct channel '9.0' (#5) - Fix test: LTS assertion checks Major instead of Minor (#6) - Fix ScopedMutex.Dispose to not leak mutex on ReleaseMutex throw (#7) - Make CommonOptions fields static readonly (#9) - Remove unused --no-progress from uninstall parsers (#10) - GarbageCollector catches Exception instead of just IOException (#11) - Capitalize 'SDK' in SdkCommandParser description (#12) - Eliminate redundant manifest reads in InstallerOrchestratorSingleton (#13) - Fix STS test error message to say 'major' not 'minor' (#14) - Make DotnetupUtilities.ExeSuffix readonly (#16) - Fix FormatBytes to use floating-point with one decimal (#17) - Remove dead ReleaseManifest variable in DotnetInstaller (#18) - Make UpdateChannel.Name immutable (#19) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
nagilson
pushed a commit
that referenced
this pull request
Aug 4, 2026
microsoft/testfx#10387 split TerminalTestReporter.Summary.cs into focused partials, adding Coverage, FlakyTests, SlowestTests and TestDiscovery. Upstream includes the folder with a glob, so nothing flagged the addition here and edits to those four files were invisible to the drift detector (microsoft/testfx#10390). Append the four paths to the dotnet-test-terminal-reporter entry, baselined at the split commit acb5bafaa2. They are appended rather than sorted in because the tracking-issue marker keys on the source index, and #7 (Summary) and #8 (TestCompletion) have open issues (dotnet#55472, dotnet#55473) that inserting would orphan. Those two keep their old baselines: their drift is real, unported feature work (retry/flaky accounting, in-process retry attribution), not just the move. Also bump source #0 (TerminalTestReporter.cs): the only upstream change since its baseline is a <remarks> comment describing upstream's partial layout, which needs no port into the SDK's single-file fork. Document both gotchas in eng/vendored-files.md. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 3328d629-8443-45a7-8ee2-97d3ce23dee5
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
1 participant
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.
No description provided.