Skip to content

[CI] declares requires python 3.9 and remove stale files - #112

Merged
pablo-estevez-lrn merged 1 commit into
masterfrom
declare-requires-python
Oct 14, 2026
Merged

pablo-estevez-lrn merged 1 commit into
masterfrom
declare-requires-python

Conversation

@pablo-estevez-lrn

@pablo-estevez-lrn pablo-estevez-lrn commented Oct 14, 2026 •

Copy link
Copy Markdown
Contributor
  • Declare a minimum supported Python version of 3.9 so pip don't update this package to newer versions if the python version is 3.8 or less(as is not tested on that version)
  • Removed setup-cfg declaration of universal, that was publishing python2 wheels, that were unusable as we are not using universal syntax.
  • Removed manifest.in that was referencing files that doesn't exists anymore, and logging a warning.

Checklist

  • Feature

  • Bug

  • Security

  • Documentation

  • ChangeLog.md updated

  • Tests added

  • All testsuites passed

  • make dist completed successfully

  • Bump package version if release coming

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

The packaging metadata and build changes consistently enforce the tested Python support range.

0 open findings

What changed in this PR

Declares Python 3.9 as the minimum supported version and cleans obsolete packaging configuration.

Changes:

  • Adds python_requires >=3.9.
  • Produces Python 3-only wheels.
  • Removes stale manifest entries and documents the support change.
File Description
setup.py Declares Python 3.9 minimum.
setup.cfg Removes universal-wheel configuration.
MANIFEST.in Removes references to nonexistent files.
Makefile Builds non-universal wheels.
ChangeLog.md Records the Python requirement.

🧠 Review effort: Balanced


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

@pablo-estevez-lrn
pablo-estevez-lrn merged commit 8403ee3 into master Oct 14, 2026
9 checks passed
@pablo-estevez-lrn
pablo-estevez-lrn deleted the declare-requires-python branch October 14, 2026 09:30
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