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

Remove py2 compat #32

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

Remove py2 compat #32

wants to merge 11 commits into from

Conversation

ralphm
Copy link
Owner

@ralphm ralphm commented May 24, 2021

Fixes #31.

@ralphm ralphm marked this pull request as draft May 24, 2021 17:40
@ralphm ralphm marked this pull request as ready for review May 25, 2021 12:07
@ralphm
Copy link
Owner Author

ralphm commented May 25, 2021

@goffi-contrib: care to take a look at this?

@goffi-contrib
Copy link

@ralphm sure, checking it now.

Copy link

@goffi-contrib goffi-contrib left a comment

Choose a reason for hiding this comment

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

the code looks fine by me, and I could successfully use it with Libervia and connect.

Copy link

@craigds craigds left a comment

Choose a reason for hiding this comment

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

The code looks 👍 to me.

I'd suggest adding to setup.py:

@micheloosterhof
Copy link

Hello! Would it be possible to merge this?
XMPP output support in Cowrie depends on this and is broken with newer versions of Twisted.

@craigds
Copy link

craigds commented May 4, 2022

@ralphm Mind merging and releasing this?

@micheloosterhof
Copy link

Maybe merge this?

@micheloosterhof
Copy link

Bump.

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.

Wokkel is not working with lastest version of Twisted (21.02)
4 participants