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

CLI polish#494

Merged
mattjohnsonpint merged 19 commits intomainfrom
pine/cli
Oct 24, 2024
Merged

CLI polish#494
mattjohnsonpint merged 19 commits intomainfrom
pine/cli

Conversation

@octref
Copy link
Contributor

@octref octref commented Oct 23, 2024

Description

Overall:

  • Uses inquirer for user input/select
  • Change the flow to:
    • SDK
    • App Name (default inferred from SDK)
    • Confirmation
  • Update all y/n with defaults
  • Contributing guide
  • npm run watch script for dev

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 marked this pull request as ready for review October 23, 2024 15:36
@octref octref requested a review from a team October 23, 2024 15:36
@mattjohnsonpint mattjohnsonpint merged commit c8f94b4 into main Oct 24, 2024
@mattjohnsonpint mattjohnsonpint deleted the pine/cli branch October 24, 2024 07:53
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