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

[green.zmq] Fix args/kwargs of socket's methods #344

Closed
wants to merge 1 commit into from
Closed

[green.zmq] Fix args/kwargs of socket's methods #344

wants to merge 1 commit into from

Conversation

gdavoian
Copy link
Contributor

@gdavoian gdavoian commented Sep 5, 2016

Make socket's methods consistent with official pyzmq library and its documentation.

Make socket's methods consistent with official pyzmq
library and its documentation.
@gdavoian gdavoian changed the title [zmq.green] Fix args/kwargs of socket's methods [green.zmq] Fix args/kwargs of socket's methods Sep 5, 2016
@ozamiatin
Copy link
Contributor

Thanks, gdavoian! My bad with arguments ...

@mchwalisz
Copy link

this should fix #376

@temoto temoto added this to the v0.21 milestone Jan 13, 2017
@temoto
Copy link
Member

temoto commented Jan 13, 2017

Thanks for reminding.

@gdavoian I hope you don't mind including few changes to zmq_test into your patch. Review here ceff431

@gdavoian
Copy link
Contributor Author

gdavoian commented Jan 14, 2017

I am ok, feel free to fix the patch. I don't mind, of course.

@codecov-io
Copy link

codecov-io commented Jan 14, 2017

Current coverage is 53.44% (diff: 84.09%)

Merging #344 into master will increase coverage by 0.17%

@@             master       #344   diff @@
==========================================
  Files            78         78          
  Lines          8902       8948    +46   
  Methods           0          0          
  Messages          0          0          
  Branches       1532       1538     +6   
==========================================
+ Hits           4742       4782    +40   
- Misses         3842       3848     +6   
  Partials        318        318          

Powered by Codecov. Last update 7595a5a...ceff431

@temoto
Copy link
Member

temoto commented Jan 14, 2017

Merged in master ceff431 will be published in next release v0.21

Have a nice day.

@temoto temoto closed this Jan 14, 2017
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.

None yet

5 participants