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

Fix #9. Emit 'open' event in createReadStream and createWriteStream. #10

Merged
merged 1 commit into from
Feb 5, 2015
Merged

Fix #9. Emit 'open' event in createReadStream and createWriteStream. #10

merged 1 commit into from
Feb 5, 2015

Conversation

eugirdor
Copy link
Contributor

@eugirdor eugirdor commented Feb 5, 2015

This fixes a bug where the 'open' event was not emitted from createReadStream and createWriteStream when they were opened.

@davidkudera
Copy link
Member

Thank you,

I just pushed updated .travis.yml file, so it would be great if you could do a rebase please.

@@ -1,7 +1,7 @@
{
"name": "fs-mock",
"description": "Simple fs mock with posix and windows file system styles",
"version": "1.1.2",
"version": "1.1.3",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please, can I ask you to revert that change? I'll do that later myself

@eugirdor
Copy link
Contributor Author

eugirdor commented Feb 5, 2015

I've made the requested changes.

@davidkudera
Copy link
Member

Just one last thing, can I ask you to squash all commits please?

This fixes a bug where the 'open' event was not emitted from createReadStream and createWriteStream when they were opened.
@eugirdor
Copy link
Contributor Author

eugirdor commented Feb 5, 2015

Done

davidkudera added a commit that referenced this pull request Feb 5, 2015
Fix #9. Emit 'open' event in createReadStream and createWriteStream.
@davidkudera davidkudera merged commit 7f9e16d into Carrooi:develop Feb 5, 2015
@davidkudera
Copy link
Member

Thanks again

@eugirdor eugirdor deleted the emit-open-event branch February 5, 2015 19:02
@eugirdor
Copy link
Contributor Author

eugirdor commented Feb 9, 2015

Any ETA on when the new version might be published?

@davidkudera
Copy link
Member

At first I wanted to wait for response to #8, but I'll probably release new version right now.

@davidkudera
Copy link
Member

done

@eugirdor
Copy link
Contributor Author

eugirdor commented Feb 9, 2015

Thanks

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