Skip to content
This repository has been archived by the owner on Apr 24, 2018. It is now read-only.

PR from 2014, rebased #79

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

PR from 2014, rebased #79

wants to merge 7 commits into from

Conversation

shaneharter
Copy link

This is the Ivoz/envoy PR, rebased as you asked 2 years ago.

- py3 doesn't need the bytes encoding
- status_code can simply be retrieved from process.poll()
- fixing for pep8 style guidelines
- ConnectedCommand.std_out and .std_err now mostly work!
@thedrow
Copy link
Collaborator

thedrow commented Sep 20, 2016

@kennethreitz Any chance for merging this and releasing it?
I can do it if you'll add me as a contributor.
This library is useful and it won't take a lot of work to bring it up to speed.

@kennethreitz
Copy link
Contributor

@thedrow done, thanks for offering! Would love to hop on skype briefly to discuss if you have time this week or next :)

send an email to me@kennethreitz.org

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants