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

add on_close_callback support for TwistedProtocolConnection #718

Closed
wants to merge 4 commits into from
Closed

add on_close_callback support for TwistedProtocolConnection #718

wants to merge 4 commits into from

Conversation

Zephor5
Copy link
Contributor

@Zephor5 Zephor5 commented Mar 10, 2016

This change is Review on Reviewable

@Zephor5 Zephor5 changed the title add on_close_callback support add on_close_callback support for TwistedProtocolConnection Mar 10, 2016
@vitaly-krugl
Copy link
Member

Thank you for your submission. Please accompany your submission with a test that verifies your change.

@codecov-io
Copy link

Current coverage is 82.54%

Merging #718 into master will not change coverage

@@           master    #718   diff @@
=====================================
  Files          19      19          
  Lines        3409    3409          
  Methods         0       0          
  Branches      536     536          
=====================================
  Hits         2814    2814          
  Misses        463     463          
  Partials      132     132          

Powered by Codecov. Last updated by 9f62cbe...5d37c24

@vitaly-krugl vitaly-krugl self-requested a review June 13, 2017 17:12
Copy link
Member

@vitaly-krugl vitaly-krugl left a comment

Choose a reason for hiding this comment

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

Thank you for your submission. Please accompany your submission with a test that verifies your change.

@Zephor5
Copy link
Contributor Author

Zephor5 commented Jun 20, 2017

these commits are not neat, close this and reopen

@Zephor5 Zephor5 closed this Jun 20, 2017
@Zephor5 Zephor5 deleted the patch-1 branch June 20, 2017 06:55
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