Skip to content

Conversation

@bigcat88
Copy link
Contributor

Following up with the changes in the ComfyUI Manager we decided to remove here pin of numpy<2 as well.

@bigcat88 bigcat88 requested a review from ltdrdata July 27, 2025 12:44
@dosubot dosubot bot added size:XS This PR changes 0-9 lines, ignoring generated files. dependencies Pull requests that update a dependency file labels Jul 27, 2025
@bigcat88 bigcat88 force-pushed the chore/remove-numpy-lesser2-pin branch from ade3787 to 1759397 Compare July 27, 2025 13:31
@bigcat88 bigcat88 force-pushed the chore/remove-numpy-lesser2-pin branch from 1759397 to c9e6a88 Compare July 27, 2025 13:42
@codecov
Copy link

codecov bot commented Jul 27, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

Flag Coverage Δ
unittests 48.03% <ø> (-0.05%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
comfy_cli/uv.py 54.50% <ø> (-0.36%) ⬇️

... and 4 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jul 28, 2025
@bigcat88 bigcat88 merged commit e56a412 into main Jul 29, 2025
14 checks passed
@bigcat88 bigcat88 deleted the chore/remove-numpy-lesser2-pin branch July 29, 2025 11:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file lgtm This PR has been approved by a maintainer size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants