Skip to content
This repository was archived by the owner on Sep 11, 2025. It is now read-only.

CLI refactor + consistent help + print enum options + validate SDK prereq#542

Merged
octref merged 12 commits intomainfrom
pine/cli-refactor
Nov 1, 2024
Merged

CLI refactor + consistent help + print enum options + validate SDK prereq#542
octref merged 12 commits intomainfrom
pine/cli-refactor

Conversation

@octref
Copy link
Contributor

@octref octref commented Oct 31, 2024

Description

  • Make all commands inherit from a BaseCommand to avoid declaring the same flags multiple times
  • Consistent padding in the help section
  • modus new: Enum options need to print possible options
  • Validate SDK prereq immediately after choosing SDK

Checklist

  • Code compiles correctly and linting passes locally
  • Tests for new functionality and regression tests for bug fixes added
  • Documentation added or updated
  • Entry added to the CHANGELOG.md file describing and linking to this PR

Thank you for your contribution to the Modus project!

@octref octref requested a review from a team October 31, 2024 07:30
@octref octref force-pushed the pine/cli-refactor branch from 9fe1ff1 to 6ed7c81 Compare October 31, 2024 07:30
@octref octref changed the title CLI refactor + consistent help CLI refactor + consistent help + print enum options + validate SDK prereq Oct 31, 2024
@octref octref merged commit c8ef559 into main Nov 1, 2024
@octref octref deleted the pine/cli-refactor branch November 1, 2024 18:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants