Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

use settings from trio for Selector waker socketpair #836

Merged
merged 4 commits into from
Dec 16, 2024

Conversation

graingert
Copy link
Collaborator

@graingert graingert commented Dec 8, 2024

Changes

Lifted from https://github.com/python-trio/trio/blob/da89ae850658fb365064cb12a47659c8a0ac44b5/src/trio/_core/_wakeup_socketpair.py#L12

Checklist

If this is a user-facing code change, like a bugfix or a new feature, please ensure that
you've fulfilled the following conditions (where applicable):

  • You've added tests (in tests/) added which would fail without your patch
  • You've updated the documentation (in docs/, in case of behavior changes or new
    features)
  • You've added a new changelog entry (in docs/versionhistory.rst).

If this is a trivial change, like a typo fix or a code reformatting, then you can ignore
these instructions.

Updating the changelog

If there are no entries after the last release, use **UNRELEASED** as the version.
If, say, your patch fixes issue #123, the entry should look like this:

- Fix big bad boo-boo in task groups
  (`#123 <https://github.com/agronholm/anyio/issues/123>`_; PR by @yourgithubaccount)

If there's no issue linked, just link to your pull request instead by updating the
changelog after you've created the PR.

@graingert graingert marked this pull request as ready for review December 15, 2024 10:51
@graingert
Copy link
Collaborator Author

not sure how to test this, also needs a changelog entry

@agronholm agronholm merged commit 3c5d4a6 into master Dec 16, 2024
16 checks passed
@agronholm agronholm deleted the graingert-patch-1 branch December 16, 2024 09:15
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants