Skip to content

spack help: Reorganize and colorize output#51484

Merged
alalazo merged 4 commits intodevelopfrom
main-argument-groups
Oct 30, 2025
Merged

spack help: Reorganize and colorize output#51484
alalazo merged 4 commits intodevelopfrom
main-argument-groups

Conversation

@tgamblin
Copy link
Copy Markdown
Member

@tgamblin tgamblin commented Oct 30, 2025

Quality of life and general command discoverability improvement. Commits intended to be rebased.

Before and after:
Screenshot 2025-10-29 at 6 42 29 PMScreenshot 2025-10-29 at 6 42 36 PM

Can't show all of spack -H / spack help --all here, but here are the options before and after:

Screenshot 2025-10-29 at 9 10 23 PMScreenshot 2025-10-29 at 9 10 34 PM

I've tried to consolidate groups and to move things around based on who's likely to use what. e.g., develop is renamed spack development and the things that had to do with other types of development are now in build and packaging.

  • Reorganize commands and options in spack help / spack -h / spack -H into more intuitive groups
  • Add nice colors to the help so it's easier to read
  • Add -e to the short help

psakievich
psakievich previously approved these changes Oct 30, 2025
Copy link
Copy Markdown
Contributor

@psakievich psakievich left a comment

Choose a reason for hiding this comment

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

LGTM. Nice improvement!

@tgamblin tgamblin force-pushed the main-argument-groups branch from b09bfa4 to 342df6b Compare October 30, 2025 08:49
@tgamblin tgamblin force-pushed the main-argument-groups branch from 342df6b to fb151a7 Compare October 30, 2025 10:15
@alalazo alalazo merged commit be21956 into develop Oct 30, 2025
33 checks passed
@alalazo alalazo deleted the main-argument-groups branch October 30, 2025 13:17
@tgamblin tgamblin added this to the v1.1.0 milestone Nov 14, 2025
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.

3 participants