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

Add options to take source and output directory. #2

Open
zanerock opened this issue Oct 24, 2018 · 0 comments
Open

Add options to take source and output directory. #2

zanerock opened this issue Oct 24, 2018 · 0 comments
Labels
good first issue Good for newcomers
Milestone

Comments

@zanerock
Copy link
Member

zanerock commented Oct 24, 2018

This issue is blocked on #3 and will be moved to the new repo once created.

No need to get fancy, just take as positional parameters for now. Assume current bash (input) and dist (output) if no arguments given. I believe the following will allow the script to take one or two arguments.

SRC=${1:-src/}
DIST=${2:-dist/}

Throw in a basic 'print usage and exit' that prints out if more than two arguments given.

Update unit tests to test:

  • defaulting values build as expected,
  • only src specified build, and
  • both specified.
@zanerock zanerock added the good first issue Good for newcomers label Oct 24, 2018
@zanerock zanerock added this to the 1.0 RC milestone Oct 24, 2018
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

1 participant