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

set options instead or argument variables #3

Closed
ritikbheda opened this issue Sep 16, 2021 · 0 comments
Closed

set options instead or argument variables #3

ritikbheda opened this issue Sep 16, 2021 · 0 comments
Assignees
Labels
good first issue Good for newcomers

Comments

@ritikbheda
Copy link

Instead of using argument variables, try using actual options which can make the work easier.
options
Over here, you need to check all the arg variables through if conditions which makes the process lengthy, error prone and time consuming. I suggest you to try using options instead which would make has pros on the above mentioned points and is much easier to use! There are many npm frameworks like commander and yargs which can do that. Follow this link which leads to use options through commander package.
Hope this helps

@tuanthanh2067 tuanthanh2067 self-assigned this Sep 17, 2021
@tuanthanh2067 tuanthanh2067 added the good first issue Good for newcomers label Sep 17, 2021
tuanthanh2067 added a commit that referenced this issue Sep 17, 2021
tuanthanh2067 added a commit that referenced this issue Sep 17, 2021
fixed #3 #4 #5 by using commander library
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants