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

partial fix for issue #979 #980

Closed
wants to merge 1 commit into from
Closed

Conversation

thefab
Copy link
Contributor

@thefab thefab commented May 11, 2016

avoid the main and blocking exception about "I/O operation on closed file"
with minimal changes and risks

avoir the main and blocking exception about "I/O operation on closed file"
with minimal changes and risks
@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 234e86c on thefab:issue979 into * on circus-tent:master*.

@k4nar
Copy link
Contributor

k4nar commented May 11, 2016

Looks great 👍 . Do you think it would be possible to add a test?

@k4nar
Copy link
Contributor

k4nar commented Jul 5, 2016

@thefab : Could you rebase it to the current master?

@k4nar k4nar mentioned this pull request Aug 12, 2016
@k4nar
Copy link
Contributor

k4nar commented Aug 12, 2016

I've opened a new PR with your commit rebased against the current master: #996

@k4nar k4nar closed this Aug 12, 2016
k4nar added a commit that referenced this pull request Aug 12, 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.

3 participants