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

allow multiple paths per script #26

Merged
merged 3 commits into from Feb 16, 2018

Conversation

ryanpineo
Copy link
Contributor

added a new paths variable that can be used in each script instead of path to allow people to specify multiple paths in 1 script without breaking backwards compatibility.

@otakup0pe
Copy link

This seems to work for me. A test would be cool tho.

@colin-nolan
Copy link

This pull request a duplicate of #17, albeit that it doesn't overload the path property (a new paths property is defined instead).

@nickhammond: It would be great if one of these requests was merged :).

@nickhammond
Copy link
Owner

Planning on merging in this one, just need to add a test for it as @otakup0pe mentioned.

@ryanpineo
Copy link
Contributor Author

@nickhammond I tried adding a test, let me know if it's good

@nickhammond
Copy link
Owner

@ryanpineo this looks great, I'll get this merged in this weekend and then bump the version. Thanks for keeping backwards compatibility as well!

@nickhammond nickhammond mentioned this pull request Feb 15, 2018
@nickhammond nickhammond merged commit 91d570f into nickhammond:master Feb 16, 2018
@nickhammond
Copy link
Owner

Merged and tagged as v0.0.5, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants