Skip to content

Improve Atmos help#959

Merged
aknysh merged 93 commits intomainfrom
feature/dev-2953-update-help-and-usageyaml-with-snapshots
Feb 13, 2025
Merged

Improve Atmos help#959
aknysh merged 93 commits intomainfrom
feature/dev-2953-update-help-and-usageyaml-with-snapshots

Conversation

@samtholiya
Copy link
Collaborator

@samtholiya samtholiya commented Jan 20, 2025

what

We would be fixing the following UX issues with help in this pr:

  • atmos about non-existent should show usage:
    image

  • Double dash in flags of atmos terraform --help and examples rendering using markdown if available.
    image

  • Fixed atmos workflow --file example.yaml bug for markdown. Now it also exits with exit code 1.
    image

  • Updated Default error logger with markdown support.
    image

  • Added custom alias help support so that alias in config should also be displayed in help.
    image

  • Updated the workflow name invalid UI
    image

  • Invalid custom command config now shows better help
    image

  • Invalid flag usage added
    image

why

  • Outputting markdown in help descriptions makes it easier to visually parse
  • Markdown stylesheet keeps formatting consistent, without each developer needing to know the style guide
  • Changed the way error that exit are displayed, to show more helpful information and usage examples, also formatted in markdown
  • Aliases were not shown in help, making their discoverability difficult

references

Summary by CodeRabbit

  • Enhanced CLI Experience

    • Error messages, usage, and help outputs across multiple Atmos commands are now displayed using a clear Markdown format. Users receive more descriptive feedback for invalid commands or flags and improved prompts during execution.
    • Specific error messages have been improved to include context and suggestions for available workflows and commands, particularly when workflows are not found or when invalid commands are issued.
    • New error messages provide clear guidance when invalid or non-existent commands are invoked, listing valid options for user convenience.
    • The command atmos terraform now specifies required subcommands and provides usage examples when invoked incorrectly.
  • Updated Documentation

    • Command usage examples and help texts for key functionality (such as for Atmos “about,” “terraform,” “workflow,” “helmfile,” and “atlantis” commands) have been refined, offering clearer guidance and structured instructions for end-user workflows.
    • New sections have been added to documentation, detailing how to apply changes to Terraform components and providing examples for using workflow commands effectively.
    • Additional help messages have been introduced for commands like atmos validate editorconfig, enhancing user understanding of available flags and options.
    • Documentation updates reflect the latest versions of Atmos and Geodesic tools, ensuring users have the most current information for their workflows.

@mergify mergify bot added the triage Needs triage label Jan 20, 2025
@samtholiya samtholiya force-pushed the feature/dev-2953-update-help-and-usageyaml-with-snapshots branch from b1741d5 to 01bcd67 Compare January 20, 2025 23:03
@osterman osterman added patch A minor, backward compatible change and removed triage Needs triage labels Jan 22, 2025
@samtholiya samtholiya force-pushed the feature/dev-2953-update-help-and-usageyaml-with-snapshots branch from 018793e to b8a0fb9 Compare January 23, 2025 21:19
@samtholiya samtholiya force-pushed the feature/dev-2953-update-help-and-usageyaml-with-snapshots branch from b8a0fb9 to bb9705b Compare January 23, 2025 21:47
@aknysh aknysh merged commit 161c074 into main Feb 13, 2025
45 checks passed
@aknysh aknysh deleted the feature/dev-2953-update-help-and-usageyaml-with-snapshots branch February 13, 2025 05:43
@mergify mergify bot removed the needs-cloudposse Needs Cloud Posse assistance label Feb 13, 2025
@github-actions
Copy link

These changes were released in v1.162.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

minor New features that do not break anything

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants