Skip to content

Conversation

@rbailey-godaddy
Copy link
Contributor

To help us get this pull request reviewed and merged quickly, please be sure to include the following items:

  • Tests (if applicable)
  • Documentation (if applicable)
  • Changelog entry
  • A full explanation here in the PR description of the work done

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • Tests
  • Other

Backward Compatibility

Is this change backward compatible with the most recently released version? Does it introduce changes which might change the user experience in any way? Does it alter the API in any way?

  • Yes (backward compatible)
  • No (breaking changes)

What's new?

  • Drop support for python 3.9; get it out of the way now
  • Clean up tox to use 3.13 for linters (vs 3.12) and don't redundantly install using pip
  • Bump pygit2 (except for python 3.10) to version that has wheel available for python 3.14
  • Remove "optional" specification my poetry hated
  • Disable pylint R0917 that now fails code that previously passed

This version went end of life on 31 October 2025; eliminating it
simplifies support for existing versions.

While we're twiddling, update tox.ini to disable pip installs of
packages since we're using poetry anyway, and bump linters to use python
3.13 (vs 3.12).
...except on python 3.10, where support has been dropped upstream and
we're already using the latest possible version.

This fixes test errors I observed where python 3.14 tried to build the
old pygit2 version itself (because no compatible wheel was available)
and died because I didn't have the compiler et al on my test system.
We haven't changed the code, but pylint apparently is pickier now than
it used to be and previously-accepted code has started failing. Disable
the offending test because we do not want to be in the business of
refactoring the code.
It doesn't work well if we tell workflow "you must use 3.12" and tell
tox "you must use 3.13". Update the workflow version to match.
While we're modernizing everything else, push the docker base image from
3.11 to 3.13. Also fix capitalization of "AS" to squelch warning
messages from docker regarding our inconsistent styling.
Duh.

Co-authored-by: Sushant Mimani <[email protected]>
@rbailey-godaddy rbailey-godaddy merged commit 774d5cf into main Nov 4, 2025
24 checks passed
@rbailey-godaddy rbailey-godaddy deleted the drop-py3.9 branch November 4, 2025 17:07
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.

5 participants