Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Dec 28, 2025

CI was failing on Python 3.8 and 3.9 because pre-commit==4.5.1 requires Python >=3.10.

Changes

  • Downgraded pre-commit from 4.5.1 to 4.3.0 in requirements-dev.txt (last version supporting Python 3.8+)
  • Added *.whl to .gitignore to prevent wheel files from being committed

The project supports Python 3.8-3.12 per pyproject.toml, but the pinned pre-commit version was incompatible with the lower bounds. Version 4.3.0 maintains the same functionality while supporting the full Python version range tested in CI.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI mentioned this pull request Dec 28, 2025
54 tasks
@coderabbitai
Copy link

coderabbitai bot commented Dec 28, 2025

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.


Comment @coderabbitai help to get the list of available commands and usage tips.

Copilot AI and others added 2 commits December 28, 2025 04:51
Co-authored-by: omsherikar <180152315+omsherikar@users.noreply.github.com>
Co-authored-by: omsherikar <180152315+omsherikar@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix quick setup problem in application Fix pre-commit version incompatibility with Python 3.8-3.9 Dec 28, 2025
Copilot AI requested a review from omsherikar December 28, 2025 04:53
@omsherikar omsherikar marked this pull request as ready for review December 30, 2025 12:49
Copilot AI review requested due to automatic review settings December 30, 2025 12:49
@github-actions github-actions bot added the bug Something isn't working label Dec 30, 2025
@omsherikar omsherikar merged commit 5557921 into feat/quick-setup Dec 30, 2025
11 checks passed
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes a Python version compatibility issue where pre-commit==4.5.1 (which requires Python >=3.10) was causing CI failures on Python 3.8 and 3.9 environments.

  • Downgraded pre-commit from 4.5.1 to 4.3.0 to support Python 3.8+
  • Added *.whl pattern to .gitignore

Reviewed changes

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

File Description
requirements-dev.txt Downgraded pre-commit version from 4.5.1 to 4.3.0 to restore compatibility with Python 3.8 and 3.9
.gitignore Added *.whl pattern to ignore wheel files

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants