-
Notifications
You must be signed in to change notification settings - Fork 87
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
added --allow-empty option to not raise an error on a missing file #88
Conversation
Thanks for your PR. I think This change needs test cases for both with and without What is the intended behaviour when the only source is missing? Should an empty file be created or not? This should be covered in the tests. |
yeah, i was thinking of the git commit |
btw, ever thought of doing the tests in cucmber? |
Aruba looks interesting. Not really a fan of the verbosity of Cucumber though. Nor do I want a big (albeit development only) dependency. |
seems to fail with |
works with --ref. test cases cover single files with --ref |
Looking pretty good. Thanks. I noticed there’s an extra space inside the file existence checks. That’ll bug me if not removed. Would you mind fixing the whitespace and then squashing these commits together? |
Thanks for the PR. |
I've added some documentation to the wiki here. |
useful if a file name depends on an environment variable: