Skip to content

Add support for Python 3.11#177

Merged
bkeryan merged 18 commits intomainfrom
users/bkeryan/python-3.11
Jan 3, 2023
Merged

Add support for Python 3.11#177
bkeryan merged 18 commits intomainfrom
users/bkeryan/python-3.11

Conversation

@bkeryan
Copy link
Copy Markdown
Collaborator

@bkeryan bkeryan commented Dec 20, 2022

What does this Pull Request accomplish?

Update PR pipeline to test with Python 3.11.

Remove typed-ast dependency. typed-ast is end-of-life and does not support Python 3.11. mypy only needs typed-ast with Python 3.7 and older, which MeasurementLink does not support.

Why should this Pull Request be merged?

Allow customers to use Python 3.11.

Fixes #154

What testing has been done?

Ran unit tests.

@bkeryan bkeryan changed the base branch from users/bkeryan/python-3.10 to main December 20, 2022 22:53
@bkeryan bkeryan force-pushed the users/bkeryan/python-3.11 branch from 0d48acc to 9e0981d Compare December 20, 2022 23:43
@bkeryan
Copy link
Copy Markdown
Collaborator Author

bkeryan commented Dec 21, 2022

In the PR workflow, I'm getting "poetry: command not found" errors on macOS.

This may be related to Gr1N/setup-poetry#33 .

@dixonjoel @mshafer-NI Do the runner images need to be updated for Python 3.11 support?

@dixonjoel
Copy link
Copy Markdown
Collaborator

In the PR workflow, I'm getting "poetry: command not found" errors on macOS.

This may be related to Gr1N/setup-poetry#33 .

@dixonjoel @mshafer-NI Do the runner images need to be updated for Python 3.11 support?

The comment thread on the issue says v8 of Gr1N has fixed this. We should try that version to see if it fixes this for us.

@bkeryan bkeryan merged commit ef26758 into main Jan 3, 2023
@bkeryan bkeryan deleted the users/bkeryan/python-3.11 branch January 3, 2023 23:04
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.

Enable Python 3.11 support in PR pipeline

3 participants