-
Notifications
You must be signed in to change notification settings - Fork 8
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
ci: Do not merge: Research Python 3.13 compatibility #337
ci: Do not merge: Research Python 3.13 compatibility #337
Conversation
Co-Authored-By: Aaron <AJ> Steers <[email protected]>
🤖 Devin AI EngineerOriginal prompt from Aaron:
I'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
Python 3.13 Compatibility Research - Final UpdateAfter thorough investigation:
Recommendation: Note: This PR should not be merged as it was created for research purposes only. |
Additional Python 3.13 Compatibility Issue FoundThe fastavro package (version 1.8.2) fails to build with Python 3.13 due to several C API changes:
This appears to be a compatibility issue with fastavro's Cython-generated C code and Python 3.13's internal C API changes. This is a more fundamental blocker than the previously identified requests-cache issue. |
Closing this research PR. The investigation revealed two key blockers for Python 3.13 compatibility:
Thank you for the opportunity to investigate this compatibility research. |
Posting here for posterity - it looks like newer versions of fastavro (starting with 1.10.0) are compatible with Python 3.13. |
Purpose
Research which CDK dependencies are not compatible with Python 3.13.
Changes
Link to Devin run
https://app.devin.ai/sessions/893114c5dc344f13af3048917ca33bf2
Requested by
Aaron
DO NOT MERGE - This is an exploratory PR to identify incompatible dependencies.