Skip to content

Conversation

@fuguiKz
Copy link
Contributor

@fuguiKz fuguiKz commented Dec 30, 2025

Context: spectreconsole/spectre.console#661 asked for clearer AddBranch guidance; CLI docs now live in this repo.

Changes:

  • Clarify AddBranch generic vs non-generic overloads on the command hierarchies page.
  • Add a tip for commands without extra args (like list): give them a dedicated settings type inheriting from the branch settings.
  • Update the CommandHierarchies demo app to use RemoteListSettings : RemoteSettings so shared options work consistently.

Verification:

  • dotnet build Spectre.Docs.slnx -c Release
  • dotnet run --project Spectre.Docs.Cli.Examples/Spectre.Docs.Cli.Examples.csproj -c Release -- 'M:Spectre.Docs.Cli.Examples.DemoApps.CommandHierarchies.Demo.RunAsync(System.String[])' remote --verbose list

@fuguiKz
Copy link
Contributor Author

fuguiKz commented Jan 1, 2026

FYI the GitHub Actions check suite is currently in an "action required" state (fork workflow approval), so CI hasn't run yet.

Could a maintainer approve and run the workflows for this PR? I ran dotnet build Spectre.Docs.slnx -c Release locally as a quick sanity check.

@phil-scott-78 phil-scott-78 merged commit e2e8369 into spectreconsole:main Jan 6, 2026
3 checks passed
@phil-scott-78
Copy link
Contributor

sorry for the delay getting to this, was out traveling for the holidays. Looks good, thanks for taking the time to improve the docs

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