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

Upgrade to aionotify 0.3.1 #15546

Merged
merged 1 commit into from
Jun 28, 2024
Merged

Upgrade to aionotify 0.3.1 #15546

merged 1 commit into from
Jun 28, 2024

Conversation

michaelosthege
Copy link
Contributor

Overview

The opentrons Python package is not yet compatible with Python 3.11. It raises errors upon import, which forces scientists to use the soon to be outdated Python 3.10.

The aionotify dependency has since been updated for Python 3.11 compatibility, so just a dependency upgrade is needed.

Test Plan

Changelog

  • Update aionotify to current version 0.3.1

Review requests

I'm not familiar with how this repository is organized, or how the Python package ending up on PyPI is built.

I hope that the CI pipeline turns green, and that maintainers can take over from there. Hopefully this remains a really easy fix for the long-standing compatibility issue #12006.

Risk assessment

@michaelosthege michaelosthege requested review from a team as code owners June 28, 2024 11:41
sfoster1 added a commit to Opentrons/buildroot that referenced this pull request Jun 28, 2024
Bump to match Opentrons/opentrons#15546 for 3.11
compatibility in the monorepo
sfoster1 added a commit to Opentrons/oe-core that referenced this pull request Jun 28, 2024
Bump to match Opentrons/opentrons#15546 for
monorepo 3.11 compat
Copy link
Member

@sfoster1 sfoster1 left a comment

Choose a reason for hiding this comment

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

Seems totally reasonable. We'll need to update aionotify's recipe in buildroot for the OT-2 and oe-core for the flex. We'll make builds there and merge this PR as soon as we can make sure they work on the machines. Thank you for the pull!

@sfoster1
Copy link
Member

I pushed https://github.com/Opentrons/opentrons/tree/edge-15546 because we need it in our remote for the robot builds, but this is the PR we'll merge

sfoster1 added a commit to Opentrons/buildroot that referenced this pull request Jun 28, 2024
Bump to match Opentrons/opentrons#15546 for 3.11
compatibility in the monorepo
sfoster1 added a commit to Opentrons/oe-core that referenced this pull request Jun 28, 2024
Bump to match Opentrons/opentrons#15546 for
monorepo 3.11 compat
@sfoster1 sfoster1 merged commit 66b23a7 into Opentrons:edge Jun 28, 2024
29 of 30 checks passed
@michaelosthege
Copy link
Contributor Author

@sfoster1 Thanks for taking over!

Did this end up in the new v7.4.0 release? I can't spot it in the changes v7.3.1...v7.4.0, but maybe that's because of how you're building releases?

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.

2 participants