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

#11738 portable gi obsolete #11739

Merged
merged 8 commits into from Nov 22, 2022
Merged

#11738 portable gi obsolete #11739

merged 8 commits into from Nov 22, 2022

Conversation

glyph
Copy link
Member

@glyph glyph commented Nov 2, 2022

Scope and purpose

Fixes #11738

Contributor Checklist:

This process applies to all pull requests - no matter how small.
Have a look at our developer documentation before submitting your Pull Request.

Below is a non-exhaustive list (as a reminder):

  • The title of the PR should describe the changes and starts with the associated issue number, like “#9782 Remove twisted.news. #1234 Brief description”.
  • A release notes news fragment file was create in src/twisted/newsfragments/ (see: Release notes fragments docs.)
  • The automated tests were updated.
  • Once all checks are green, request a review by leaving a comment that contains exactly the string please review.
    Our bot will trigger the review process, by applying the pending review label
    and requesting a review from the Twisted dev team.

@glyph glyph changed the title 11738 portable gi obsolete #11738 portable gi obsolete Nov 2, 2022
@glyph glyph marked this pull request as ready for review November 2, 2022 21:38
@chevah-robot chevah-robot requested a review from a team November 2, 2022 21:38
@glyph
Copy link
Member Author

glyph commented Nov 2, 2022

io_add_watch has been supported with sockets on windows since 2016:

https://pygobject.readthedocs.io/en/latest/changelog.html?highlight=io_add_watch#id45

@adiroiban adiroiban requested a review from a team November 2, 2022 23:05
Co-authored-by: Adi Roiban <adiroiban@gmail.com>
@glyph
Copy link
Member Author

glyph commented Nov 3, 2022

Sigh. The test failures are just codecov falling over again. I really want to get on a pure github actions workflow for coverage

@exarkun
Copy link
Member

exarkun commented Nov 3, 2022

Sigh. The test failures are just codecov falling over again. I really want to get on a pure github actions workflow for coverage

I'm trying to observe this but I can't figure it out. Where do I look to see spurious codecov failures for this PR?

@adiroiban
Copy link
Member

The failures can be seen here - https://github.com/twisted/twisted/actions/runs/3381705937/jobs/5615891650#step:13:38

The errors are generated while trying to upload the reports to Codecov "cloud".

Copy link
Member

@adiroiban adiroiban left a comment

Choose a reason for hiding this comment

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

With the tests green, I think we are good to go, assuming the release notes are updated as suggested by JP :)

Thanks!

glyph and others added 2 commits November 21, 2022 15:23
@glyph glyph merged commit 602718a into trunk Nov 22, 2022
@glyph glyph deleted the 11738-portable-gi-obsolete branch November 22, 2022 00:16
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.

PortableGIReactor is obsolete
4 participants