Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade click dependency #34

Merged
merged 10 commits into from
May 2, 2024
Merged

Upgrade click dependency #34

merged 10 commits into from
May 2, 2024

Conversation

MarcAndre-Rivet
Copy link
Collaborator

@MarcAndre-Rivet MarcAndre-Rivet commented May 1, 2024

Updates click dependency.

  • This allows downstream projects requiring celery>=5.3 to continue using this project, as Celery>=5.3 requires click>=8.1.2 / causing an incompatible version matchup in Poetry
  • Dropping py36 should not be of consequence as it's been past EOL for over a year now
  • Testing against more recent versions of Python (3.10+) required an update to pytest as well

@MarcAndre-Rivet
Copy link
Collaborator Author

This gh rule will need updating as this will never be executed
image

@tacrocha tacrocha self-requested a review May 1, 2024 20:39
tacrocha
tacrocha previously approved these changes May 1, 2024
@MarcAndre-Rivet MarcAndre-Rivet requested a review from a team as a code owner May 2, 2024 12:57
@MarcAndre-Rivet MarcAndre-Rivet requested review from tacrocha and removed request for a team May 2, 2024 12:57
@@ -12,13 +12,13 @@ authors = [
license = "MIT"

[tool.poetry.dependencies]
python = "^3.6"
python = ">=3.7,<3.12"
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Updating click drops py3.6 support. The project also doesn't work with py3.12 as per test run as-is, hence the updated version range. We can always revisit as necessary.

{ "os": "ubuntu-latest", "python": "3.8" },
{ "os": "ubuntu-latest", "python": "3.9" },
{ "os": "ubuntu-latest", "python": "3.10" },
{ "os": "ubuntu-latest", "python": "3.11" },
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

testing the full range of versions supported

@MarcAndre-Rivet MarcAndre-Rivet changed the title Upgrade click (and side-effects) Upgrade click dependency May 2, 2024
Copy link
Contributor

@codingtony-candid codingtony-candid left a comment

Choose a reason for hiding this comment

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

lgtm

@MarcAndre-Rivet MarcAndre-Rivet merged commit e2ac4a3 into master May 2, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants