-
-
Notifications
You must be signed in to change notification settings - Fork 143
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
refactor(waku): prettify cli using commander.js #339
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. Latest deployment of this branch, based on commit ba230a0:
|
Yeah, I was thinking about it too. But, I have a few concerns:
|
There are some options: arg: https://github.com/huozhi/bunchee/blob/main/src/bin/index.ts |
Thanks. I would like to hold this for the moment, and revisit it in the future. There might be more hidden requirements in cli.ts and we can't decide it now. |
This limitation is gone with #355. |
I'm still interested in using commander.js or some alternatives (maybe commander.js is fine.) Maybe some time later, maybe before v1-alpha. Let's start over anyway. |
#1319 should be a follow-up. |
now the CLI has growing complex, I think we need a standard library to handle the thing