Skip to content

build(deps): prevent update to click@8.2.0 #1245

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

Merged

Conversation

codejedi365
Copy link
Contributor

@codejedi365 codejedi365 commented May 16, 2025

Purpose

  • Prevent breaking change to test cases from click update

Rationale

click 8.2 has a breaking change in the test suite where it removes a parameter from the cli runner. There is additional deprecation warnings included as well so for now since it will break our tests limit this to 8.1.0 only.

How did you test?

Verified the CI passed both the mypy lint step and the E2E tests worked.


PR Completion Checklist

  • Reviewed & followed the Contributor Guidelines

  • Changes Implemented & Validation pipeline succeeds

  • Commits follow the Conventional Commits standard
    and are separated into the proper commit type and scope (recommended order: test, build, feat/fix, docs)

  • N/A Appropriate Unit tests added/updated

  • N/A Appropriate End-to-End tests added/updated

  • N/A Appropriate Documentation added/updated and syntax validated for sphinx build (see Contributor Guidelines)

click 8.2 has a breaking change in the test suite where it removes
a parameter from the cli runner. There is additional deprecation
warnings included as well so for now since it will break our tests
limit this to 8.1.0 only.
@codejedi365 codejedi365 merged commit 4aa6a6e into python-semantic-release:master May 16, 2025
12 checks passed
@codejedi365 codejedi365 deleted the build/fix-click-errors branch May 16, 2025 04:50
@codejedi365
Copy link
Contributor Author

🎉 This PR has been published as part of v10.0.0 🎉

You can find more information about this release on the GitHub Releases page.

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.

1 participant