Skip to content

Drop support for Python versions older than 3.12 - #101

Merged
martinmarques merged 1 commit into
mainfrom
dev/bar-1568
Aug 13, 2026
Merged

Drop support for Python versions older than 3.12#101
martinmarques merged 1 commit into
mainfrom
dev/bar-1568

Conversation

@joao00001

Copy link
Copy Markdown
Contributor

pg-backup-api now requires Python 3.12 or higher. This aligns with the broader project's move to edb-python312 and enables a minor release cycle. CI matrices and tox are updated to cover Python 3.12, 3.13, and 3.14. The change is documented in news.yml under the upcoming 2.3.0 release.

References: BAR-1568

@joao00001 joao00001 self-assigned this Aug 11, 2026
@joao00001
joao00001 force-pushed the dev/bar-1568 branch 3 times, most recently from 2c02198 to 6e8cd25 Compare August 11, 2026 14:13
@joao00001
joao00001 requested a lite review from Copilot August 11, 2026 14:14

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This pull request updates pg-backup-api to require Python 3.12+, aligning packaging metadata, tox environments, CI matrices, and tests with the new supported Python baseline.

Changes:

  • Updated tox and GitHub Actions matrices to test Python 3.12–3.14.
  • Raised package python_requires to >=3.12 and removed Python-3.7-specific test conditionals.
  • Adjusted CLI help-output tests to be resilient to argparse wrapping differences across newer Python versions.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
pg_backup_api/tox.ini Drops old Python env factors; runs tox envs for 3.12–3.14.
pg_backup_api/setup.py Raises python_requires to >=3.12.
pg_backup_api/pg_backup_api/tests/test_utility_controller.py Removes legacy Python/Flask version-conditional assertions tied to EOL Python versions.
pg_backup_api/pg_backup_api/tests/test_main.py Normalizes CLI help output to reduce version-specific argparse wrapping differences.
pg_backup_api/news.yml Documents the Python support drop under the upcoming 2.3.0 notes.
.github/workflows/tests.yml Updates CI Python matrix to 3.12–3.14.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread pg_backup_api/pg_backup_api/tests/test_main.py
Comment thread pg_backup_api/setup.py
pg-backup-api now requires Python 3.12 or higher. This aligns with
the broader project's move to edb-python312 and enables a minor
release cycle. CI matrices and tox are updated to cover Python 3.12,
3.13, and 3.14. The change is documented in news.yml under the
upcoming 2.3.0 release.

References: BAR-1568
Signed-off-by: Joao Detomini <joao.detomini@enterprisedb.com>
@joao00001

Copy link
Copy Markdown
Contributor Author

To be aligned with barman 3.20, which already tests against Python 3.12, 3.13, and 3.14, I extended the CI matrix here to cover the same versions. This surfaced two pre-existing test issues (a distutils removal in 3.12+ and an argparse wrapping change in 3.13+), which were fixed as part of this PR. IMO it shouldn't be a bit deal keeping this changes.

@joao00001
joao00001 marked this pull request as ready for review August 11, 2026 14:27
@joao00001
joao00001 requested a review from barthisrael August 11, 2026 14:31
@martinmarques
martinmarques requested a lite review from Copilot August 13, 2026 13:07

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated no new comments.

@martinmarques
martinmarques merged commit 0305891 into main Aug 13, 2026
17 checks passed
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.

3 participants