Skip to content

issue-551: Restricting python version and removing pkg_resources#552

Open
EmilyPascua wants to merge 3 commits intomasterfrom
issue-551
Open

issue-551: Restricting python version and removing pkg_resources#552
EmilyPascua wants to merge 3 commits intomasterfrom
issue-551

Conversation

@EmilyPascua
Copy link

Pull request to address #551 ; this removes support for python 3.7 which has already reached eol. Restricting python version to >= 3.10 < 3.11.

@EmilyPascua EmilyPascua requested a review from nttoole March 5, 2026 18:14
@EmilyPascua EmilyPascua requested review from a team as code owners March 5, 2026 18:14
@nttoole
Copy link
Contributor

nttoole commented Mar 10, 2026

@EmilyPascua Overall looks good, tho I noticed a lot of the pretty alignment from pyproject.toml was removed? I tend to be pro-alignment so was wondering if it was violating some convention?

@EmilyPascua
Copy link
Author

@EmilyPascua Overall looks good, tho I noticed a lot of the pretty alignment from pyproject.toml was removed? I tend to be pro-alignment so was wondering if it was violating some convention?

@nttoole Thanks Nicko! The auto formatter on my IDE updated it, but I can revert it back. Will push a change for this.

@EmilyPascua
Copy link
Author

EmilyPascua commented Mar 26, 2026

Tests failing due to still expecting other python versions; though sphinx is failing due to using pkg_resources.

  • Need sphinx version update for docs.
  • Deprecate testing python versions > 3.10.

fail-fast: false
matrix:
python-version: ["3.7", "3.8", "3.9", "3.10"]
python-version: ["3.10"]
Copy link
Contributor

Choose a reason for hiding this comment

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

Should 3.11 be included at this point, or no?


def getDefaultSchema(): # noqa
return pkg_resources.resource_filename("ait.core", "data/cmd_schema.json")
return files("ait.core").joinpath("data/cmd_schema.json")
Copy link
Contributor

Choose a reason for hiding this comment

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

Do these need a str() call to ensure return type is maintained?

3.8.15
3.9.15
3.10.8
3.10.20
Copy link
Contributor

Choose a reason for hiding this comment

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

Similar question re 3.11

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