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

Update supported Python versions #92

Merged
merged 2 commits into from
Mar 13, 2024
Merged

Conversation

jonafato
Copy link
Collaborator

@jonafato jonafato commented Mar 8, 2024

This PR also makes a few logical changes (mostly regarding test running) in order to get tests to pass under Python 3.12.

  • Remove Python 3.7 from the test matrix and list of supported versions in setup.py, as it reached end of life on 2023-06-27
  • Add Python 3.12 to the list of tested / supported versions as the current major version
  • Replace usage of python setup.py test and the PyTest test command with a more direct invocation of pytest in tox.ini
  • Replace usage of pkg_resources.get_distribution with importlib.metadata to get the installed version number for vladiate

- Remove Python 3.7 from the test matrix and list of supported versions
  in `setup.py`, as it reached end of life on 2023-06-27
- Add Python 3.12 to the list of tested / supported versions as the
  current major version
@jonafato jonafato marked this pull request as draft March 8, 2024 22:00
- Replace usage of `python setup.py test` and the `PyTest` test command
  with a more direct invocation of `pytest` in `tox.ini`
- Replace usage of `pkg_resources.get_distribution` with
  `importlib.metadata` to get the installed version number for
  `vladiate`
@jonafato jonafato marked this pull request as ready for review March 9, 2024 02:20
@jonafato
Copy link
Collaborator Author

@di does this seem reasonable? I'm sure there's more to be done to modernize the packaging config here, but my main goal was to get tests running for 3.12.

Copy link
Owner

@di di left a comment

Choose a reason for hiding this comment

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

LGTM!

@di di merged commit 48f92e1 into di:master Mar 13, 2024
6 checks passed
@jonafato jonafato deleted the update-python-versions branch March 13, 2024 17:54
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.

None yet

2 participants