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

Source & output folders can now be set. cli help output formatted. Added more test snippets #2

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

Conversation

andfaulkner
Copy link

User can now set source & output folders with new command line arguments (--src and --out).
Formatted cli help output to be multiline.
Added a few more test snippets (in output/test folder).
Switched string concat-based file path building with Node's native path.join function (much cleaner and less error-prone).

@james2doyle
Copy link
Owner

This seems like it could be useful. I would say you will need to update the README to reflect the changes, also set some defaults for the arguments that match this base repo. Right now people are forced to put arguments, and if they don't it just errors out without any info.

@andfaulkner
Copy link
Author

Cool, I'll do it some time in the next few days.

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