Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Restructure project commands to meet guidelines #1266

Open
wants to merge 2 commits into
base: next
Choose a base branch
from

Conversation

brandenrodgers
Copy link
Contributor

@brandenrodgers brandenrodgers commented Nov 22, 2024

Description and Context

hs project

  • Update description to: "Commands for managing projects..."
  • Remove all flags except for the --help flag because they don't do anything
  • Support hs projects as a plural alias

hs project create

  • Add a period to the description
  • use --dest instead of location
  • use kebab case in all flags
  • Remove --version flag because it's unnecessary

hs project add

  • Add a period to the description
  • Remove --version flag because it's unnecessary

hs project watch

  • Add a period to the description
  • Remove --version flag because it's unnecessary
  • Remove path positional (this should be run from within a project)

hs project dev

  • Add a period to the description
  • Remove --version flag because it's unnecessary
  • Remove account positional

hs project upload

  • Add a period to the description
  • Remove --version flag because it's unnecessary
  • Remove path, forceCreate, and message positionals
  • Support -m alias for message
  • use kebab case for arguments

hs project deploy

  • Add a period to the description
  • Remove --version flag because it's unnecessary
  • use kebab case for flags

hs project logs

  • Add a period to the description
  • Remove --version flag because it's unnecessary

hs project list-builds

  • Add a period to the description
  • Remove --version flag because it's unnecessary
  • Remove path positional. Support --project flag, but fall back to the project in the CWD when not provided

hs project download

  • Add a period to the description
  • Remove --version flag because it's unnecessary
  • project should be a flag, not a positional
  • use kebab for all flags

hs project open

  • Add a period to the description
  • Remove --version flag because it's unnecessary
  • project should be a flag, not a positional

hs project migrate-app

  • Add a period to the description
  • Remove --version flag because it's unnecessary
  • location should be switched to dest

hs project clone-app

  • Add a period to the description
  • Remove --version flag because it's unnecessary
  • location should be switched to dest

Screenshots

TODO

Who to Notify

@brandenrodgers brandenrodgers changed the base branch from main to next November 22, 2024 21:35
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.

1 participant