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

Use proxyquire to mock fs object ... #7

Closed
wants to merge 1 commit into from

Conversation

ta2edchimp
Copy link
Collaborator

instead of using mock-fs.

This should fix the issue shown in #6

@kentcdodds
Copy link
Owner

Huh... I wonder why this didn't kick off a travis build... I'll pull it down locally and give it a look. Thanks!

@ta2edchimp
Copy link
Collaborator Author

I think travis only kicks in on PRs to the master branch, I compared against and created the PR towards your ava-bug branch.

@kentcdodds
Copy link
Owner

Ah! That makes sense. Do you mind making this same PR to master? I haven't had a chance to pull it down yet. Thanks 👍

@ta2edchimp
Copy link
Collaborator Author

See #8.

Strange, this pr got its checks through travis now...

@ta2edchimp ta2edchimp closed this Mar 17, 2016
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