[proxy] web.archive.org← back | site home | direct (HTTPS) ↗ | proxy home | ◑ dark◐ light
/ cpython Public

Conversation

Copy link
Member

1st1 commented Nov 17, 2017

1st1 requested a review from asvetlov November 17, 2017 23:30
1st1 changed the title Support pathlib.Path in create_unix_connection; sock arg should be optional bpo-32066: Support pathlib.Path in create_unix_connection; sock arg should be optional Nov 17, 2017
Copy link
Contributor

asvetlov left a comment

Choose a reason for hiding this comment

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

Looks good but asyncio-eventloop.rst should reflect accepted pathlib.Path parameter for create_unix_connection and create_unix_server IMHO

Copy link

A Python core developer has requested some changes be made to your pull request before we can consider merging it. If you could please address their requests along with any other requests in other reviews from core developers that would be appreciated.

Once you have made the requested changes, please leave a comment on this pull request containing the phrase I have made the requested changes; please review again. I will then notify any core developers who have left a review that you're ready for them to take another look at this pull request.

Copy link
Member Author

1st1 commented Nov 18, 2017

I have made the requested changes; please review again

Copy link

Thanks for making the requested changes!

@asvetlov: please review the changes made to this pull request.

1st1 merged commit 423fd36 into python:master Nov 20, 2017
1st1 deleted the fix_unix_con branch November 20, 2017 22:26
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.

4 participants