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

make output folder in test folder #330

Merged
merged 1 commit into from May 11, 2015
Merged

make output folder in test folder #330

merged 1 commit into from May 11, 2015

Conversation

eddiemonge
Copy link
Contributor

Match the output folder relative location created by other for the cli tests.
Other creates test/other/output
CLI now creates test/cli/output

Better would be using something like https://www.npmjs.com/package/tmp so the created folders aren't even in the project

Match the output folder relative location created by other for the cli tests.
Other creates test/other/output
CLI now creates test/cli/output
@coveralls
Copy link

Coverage Status

Coverage remained the same at 97.91% when pulling 3fb08ca on eddiemonge:outputdir into c7bc4cc on gotwarlost:master.

davglass added a commit that referenced this pull request May 11, 2015
make output folder in test folder
@davglass davglass merged commit 6b29096 into gotwarlost:master May 11, 2015
@eddiemonge eddiemonge deleted the outputdir branch May 11, 2015 19:33
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.

None yet

3 participants