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

Test Kafka 0.8.2.2 using Python 3.10 in the meantime #161

Merged
merged 6 commits into from
Mar 9, 2024
Merged

Conversation

wbarnha
Copy link
Owner

@wbarnha wbarnha commented Mar 8, 2024

Since 3.12 can't support Kafka v0.8.2.2 at the moment, but we know older versions of Python can, let's try to use an earlier version just to make sure we don't break anything for users until we document it.

@wbarnha
Copy link
Owner Author

wbarnha commented Mar 8, 2024

Definitely will need to go back and revert this PR someday, I don't want to make the CI/CD too messy.

@wbarnha
Copy link
Owner Author

wbarnha commented Mar 9, 2024

Passes on 3.10 but on kafka-python I managed to get it passing on 3.11. Something else is going on here.

@wbarnha wbarnha closed this Mar 9, 2024
@wbarnha wbarnha reopened this Mar 9, 2024
@wbarnha wbarnha changed the title Test Kafka 0.8.2.2 using Python 3.11 in the meantime Test Kafka 0.8.2.2 using Python 3.10 in the meantime Mar 9, 2024
@wbarnha
Copy link
Owner Author

wbarnha commented Mar 9, 2024

So something must've been introduced in 3.11 that broke compatibility with 0.8.2.2.

@wbarnha wbarnha self-assigned this Mar 9, 2024
@wbarnha wbarnha merged commit e762321 into master Mar 9, 2024
22 of 23 checks passed
@wbarnha wbarnha deleted the keep0822 branch March 9, 2024 00:50
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.

1 participant