Skip to content

ci: temporarily install click<8.3.0#2288

Merged
anakin87 merged 2 commits intomainfrom
hatch-pin-click
Sep 19, 2025
Merged

ci: temporarily install click<8.3.0#2288
anakin87 merged 2 commits intomainfrom
hatch-pin-click

Conversation

@anakin87
Copy link
Copy Markdown
Member

@anakin87 anakin87 commented Sep 19, 2025

Related Issues

Proposed Changes:

  • temporarily install click<8.3.0 in the CI, along with hatch

How did you test it?

CI

Notes for the reviewer

Other CI failures are not related to this PR and were already present.
See for example meta-llama:

Checklist

@anakin87 anakin87 changed the title temporarily install click<8.3.0 chore: temporarily install click<8.3.0 Sep 19, 2025
@anakin87 anakin87 changed the title chore: temporarily install click<8.3.0 ci: temporarily install click<8.3.0 Sep 19, 2025
@anakin87 anakin87 self-assigned this Sep 19, 2025
@anakin87 anakin87 marked this pull request as ready for review September 19, 2025 07:50
@anakin87 anakin87 requested a review from a team as a code owner September 19, 2025 07:50
@anakin87 anakin87 requested review from julian-risch and removed request for a team September 19, 2025 07:50
Copy link
Copy Markdown
Member

@julian-risch julian-risch left a comment

Choose a reason for hiding this comment

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

LGTM! 👍 Small fix in the NvidiaChatGenerator test looks good too.

@anakin87 anakin87 merged commit 635aeb7 into main Sep 19, 2025
177 of 188 checks passed
@anakin87 anakin87 deleted the hatch-pin-click branch September 19, 2025 08:09
@anakin87 anakin87 mentioned this pull request Sep 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment