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

Release 1.0.4 #18

Merged
merged 7 commits into from Aug 10, 2023
Merged

Release 1.0.4 #18

merged 7 commits into from Aug 10, 2023

Conversation

adiroiban
Copy link
Member

@adiroiban adiroiban commented Aug 9, 2023

Scope and purpose

To get support for python 3.12

How to check

Check the README rendering on https://github.com/twisted/twisted-iocpsupport/tree/release-1.0.4

Check pypi pre-release https://pypi.org/project/twisted-iocpsupport/1.0.4rc1/

@adiroiban
Copy link
Member Author

I think this is ready for review

needs-review

@adiroiban adiroiban requested a review from a team August 9, 2023 22:41
@adiroiban
Copy link
Member Author

@pmisik if you have time, you can try the release candidate with py3.12

I guess that once this is released, we will need to update twisted/twisted

@adiroiban
Copy link
Member Author

Thanks @glyph for the review.

Note that the current release is done via the already existing fixed GitHub API token .

I don't have admin permissions for PyPi for twisted-iocpsupport

@adiroiban
Copy link
Member Author

@adiroiban adiroiban merged commit 3615ed0 into default Aug 10, 2023
10 checks passed
@adiroiban adiroiban deleted the release-1.0.4 branch August 10, 2023 10:26
@glyph
Copy link
Member

glyph commented Aug 10, 2023

Thanks @glyph for the review.

Note that the current release is done via the already existing fixed GitHub API token .

I don't have admin permissions for PyPi for twisted-iocpsupport

I rolled out permissions a bit slowly with the org feature on PyPI, but I've started pruning non-org contributors and giving the core team "owner" status on more projects now. So you should have admin permissions now.

Do you need them to address any current concerns though? My understanding was that the automated PyPI releases on this repo are pretty thorough

@adiroiban
Copy link
Member Author

Thanks.

Do you need them to address any current concerns though? My understanding was that the automated PyPI releases on this repo are pretty thorough

The current API token works and the release was done.

It's just that if it would have stopped working, I would have not had permission to generate a new token.

@pmisik
Copy link
Contributor

pmisik commented Aug 15, 2023

@pmisik if you have time, you can try the release candidate with py3.12
I guess that once this is released, we will need to update twisted/twisted

@adiroiban thank you for incredible fast response.
I tried and reported another issue twisted/twisted#11908
Reported issue generates huge deprecationWarning log for buildbot.
But in the end it seems it hang. At the moment I have no idea why.
In first stage I try to solve deprecation warning issue.

@adiroiban
Copy link
Member Author

But in the end it seems it hang. At the moment I have no idea why.

What hangs ?

I think that you should be able to run buildbot on Windows without the iocp reactor.
The select reactor should work just fine.


I agree that the deprecationWarning should be fixed in twisted/twisted

Regards

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants