Skip to content

Test Snapshot Completion Automation#7

Open
nagilson wants to merge 3 commits into
nagilson-clean-merge-jjonescz-sprint-clnfrom
nagilson-snapshot-completions-undone
Open

Test Snapshot Completion Automation#7
nagilson wants to merge 3 commits into
nagilson-clean-merge-jjonescz-sprint-clnfrom
nagilson-snapshot-completions-undone

Conversation

@nagilson

Copy link
Copy Markdown
Owner

No description provided.

@nagilson nagilson changed the base branch from main to release/10.0.1xx December 10, 2025 19:28
@nagilson nagilson changed the base branch from release/10.0.1xx to nagilson-clean-merge-jjonescz-sprint-cln December 10, 2025 19:37
@nagilson

Copy link
Copy Markdown
Owner Author

/fixcompletions

1 similar comment
@nagilson

Copy link
Copy Markdown
Owner Author

/fixcompletions

@github-actions

Copy link
Copy Markdown

❌ Failed to update completion snapshots. Please check the workflow logs for details.

@nagilson

Copy link
Copy Markdown
Owner Author

/fixcompletions

@github-actions

Copy link
Copy Markdown

ℹ️ No completion snapshot files needed to be updated.

@nagilson

Copy link
Copy Markdown
Owner Author

/fixcompletions

@github-actions

Copy link
Copy Markdown

▶️ CLI completion snapshot update started. Track progress in this workflow run.

@github-actions

Copy link
Copy Markdown

ℹ️ No completion snapshot files needed to be updated. Review the workflow run.

@nagilson

Copy link
Copy Markdown
Owner Author

/fixcompletions

1 similar comment
@nagilson

Copy link
Copy Markdown
Owner Author

/fixcompletions

@github-actions

Copy link
Copy Markdown

▶️ CLI completion snapshot update started. Track progress in this workflow run.

@github-actions

Copy link
Copy Markdown

✅ 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 :)
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 (dotnet#18)
- Include component type in error/warning messages (#11)
- Remove component spec from runtime update (dotnet#12)
- Remove GetInstallations wrapper (#6)
- ScopedMutex throws TimeoutException on failure (dotnet#22)
- Make InstallResult.Install non-nullable (dotnet#17)
- Remove ComponentFilesExist shortcut (dotnet#19)
- Wire up InstallSource enum (dotnet#21)
- ListCommand: enum types, narrowed mutex, read-only verify, spacing (#3,#4,#5,#1)
- Show global.json path in list output (dotnet#15)
- Add doc comments to ListData types (#7)
- Move SourceOption to CommonOptions, remove alias fields (#9)
- Remove ResolveChannelFromGlobalJson test hook wrapper (dotnet#14)
- Root 'dotnetup update' updates all components (dotnet#16)
- GlobalJsonChannelResolver: support rollForward policy (dotnet#13)
- Error when installing to unmanaged dotnet root (dotnet#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 (dotnet#12)
- Eliminate redundant manifest reads in InstallerOrchestratorSingleton (dotnet#13)
- Fix STS test error message to say 'major' not 'minor' (dotnet#14)
- Make DotnetupUtilities.ExeSuffix readonly (dotnet#16)
- Fix FormatBytes to use floating-point with one decimal (dotnet#17)
- Remove dead ReleaseManifest variable in DotnetInstaller (dotnet#18)
- Make UpdateChannel.Name immutable (dotnet#19)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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.

1 participant