Skip to content

Conversation

@rrzaripov
Copy link

This closes issue #113 for me. But the code review is needed, possibly this solution is too slow.

@codecov-io
Copy link

Current coverage is 96.33%

Merging #122 into master will not affect coverage as of fc1378a

@@            master    #122   diff @@
======================================
  Files           11      11       
  Stmts          982     983     +1
  Branches         0       0       
  Methods          0       0       
======================================
+ Hit            946     947     +1
  Partial          0       0       
  Missed          36      36       

Review entire Coverage Diff as of fc1378a

Powered by Codecov. Updated on successful CI builds.

@dnozay dnozay changed the title This closes #113 add flush() to fix #113 Feb 12, 2016
@dnozay
Copy link
Member

dnozay commented Mar 2, 2016

@rrzaripov any update? If I could get your help, maybe I can close this issue sooner.

@rrzaripov rrzaripov closed this Jul 4, 2016
@rrzaripov rrzaripov deleted the flush-stream-xml-test-result branch July 4, 2016 18:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants