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

move from minimist to commander #134

Merged
merged 1 commit into from
Nov 10, 2023
Merged

move from minimist to commander #134

merged 1 commit into from
Nov 10, 2023

Conversation

mansona
Copy link
Member

@mansona mansona commented Nov 10, 2023

This gives us a lot more feedback when running the scripts in this repo if we pass the wrong args, it also gives us a nice --help functionality πŸŽ‰

➜  ember-jsonapi-docs git:(commander) βœ— ./index.js --help                   
Usage: index [options]

Options:
  -p, --projects <project...>  the projects that you want to run this for (choices: "ember", "ember-data")
  -v, --version <version>      project version
  -a, --all                    process all versions (this will take a long time)
  -c, --clean                  clean (not sure what this does)
  -h, --help                   display help for command

➜  ember-jsonapi-docs git:(commander) ./generate-local.js --help
Usage: generate-local [options]

Options:
  -p, --project <project>  the project that you want to run this for (choices: "ember", "ember-data")
  -v, --version <version>  project version
  -h, --help               display help for command

@mansona mansona requested a review from jaredgalanis November 10, 2023 20:59
Copy link
Member

@jaredgalanis jaredgalanis left a comment

Choose a reason for hiding this comment

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

Nice! πŸ‘

@mansona mansona merged commit 5b2c728 into master Nov 10, 2023
2 checks passed
@mansona mansona deleted the commander branch November 10, 2023 22:47
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.

2 participants