Skip to content

chore: Uncap requires-python in package metadata#3293

Merged
mergify[bot] merged 1 commit intoinstructlab:mainfrom
booxter:uncap-python
Apr 19, 2025
Merged

chore: Uncap requires-python in package metadata#3293
mergify[bot] merged 1 commit intoinstructlab:mainfrom
booxter:uncap-python

Conversation

@booxter
Copy link
Contributor

@booxter booxter commented Apr 16, 2025

This blocks people from trying the package out on newer Python versions
without hacks. A good expose on why upper caps are in general a bad idea
and should be avoided can be found at:

https://iscinumpy.dev/post/bound-version-constraints/

(Look for 'Pinning the Python version is special' title.)

Signed-off-by: Ihar Hrachyshka ihar.hrachyshka@gmail.com

Checklist:

  • Commit Message Formatting: Commit titles and messages follow guidelines in the
    conventional commits.
  • Changelog updated with breaking and/or notable changes for the next minor release.
  • Documentation has been updated, if necessary.
  • Unit tests have been added, if necessary.
  • Functional tests have been added, if necessary.
  • E2E Workflow tests have been added, if necessary.

This blocks people from trying the package out on newer Python versions
without hacks. A good expose on why upper caps are in general a bad idea
and should be avoided can be found at:

https://iscinumpy.dev/post/bound-version-constraints/

(Look for 'Pinning the Python version is special' title.)

Signed-off-by: Ihar Hrachyshka <ihar.hrachyshka@gmail.com>
@mergify mergify bot added the one-approval PR has one approval from a maintainer label Apr 17, 2025
@booxter booxter requested a review from a team April 18, 2025 15:05
@mergify mergify bot removed the one-approval PR has one approval from a maintainer label Apr 18, 2025
@booxter booxter removed the request for review from RobotSail April 19, 2025 01:41
@mergify mergify bot merged commit 7a3d6d3 into instructlab:main Apr 19, 2025
27 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