Skip to content

Conversation

@fengsi
Copy link
Contributor

@fengsi fengsi commented Oct 15, 2025

Followup PR for #330.

@dosubot dosubot bot added size:XS This PR changes 0-9 lines, ignoring generated files. bug Something isn't working labels Oct 15, 2025
@codecov
Copy link

codecov bot commented Oct 15, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

@@           Coverage Diff           @@
##             main     #332   +/-   ##
=======================================
  Coverage   52.36%   52.36%           
=======================================
  Files          32       32           
  Lines        3516     3516           
=======================================
  Hits         1841     1841           
  Misses       1675     1675           
Files with missing lines Coverage Δ
comfy_cli/command/models/models.py 40.00% <100.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@fengsi fengsi force-pushed the si/fix-model-list branch from 91a25b9 to e9836c8 Compare October 16, 2025 05:35
Copy link
Contributor

@bigcat88 bigcat88 left a comment

Choose a reason for hiding this comment

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

looks good, merging

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Oct 16, 2025
@bigcat88 bigcat88 merged commit 81bf867 into Comfy-Org:main Oct 16, 2025
11 checks passed
@fengsi fengsi deleted the si/fix-model-list branch October 16, 2025 18:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working 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.

2 participants