Skip to content

Set minimal supported python to version 3.10#349

Merged
bigcat88 merged 1 commit intomainfrom
chore/drop-python3.9-2
Jan 7, 2026
Merged

Set minimal supported python to version 3.10#349
bigcat88 merged 1 commit intomainfrom
chore/drop-python3.9-2

Conversation

@bigcat88
Copy link
Contributor

@bigcat88 bigcat88 commented Jan 7, 2026

No description provided.

@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. enhancement New feature or request labels Jan 7, 2026
@codecov
Copy link

codecov bot commented Jan 7, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

@@            Coverage Diff             @@
##             main     #349      +/-   ##
==========================================
- Coverage   52.33%   52.23%   -0.11%     
==========================================
  Files          32       32              
  Lines        3527     3519       -8     
==========================================
- Hits         1846     1838       -8     
  Misses       1681     1681              
Files with missing lines Coverage Δ
comfy_cli/cmdline.py 49.39% <100.00%> (ø)
comfy_cli/command/custom_nodes/command.py 43.19% <100.00%> (ø)
comfy_cli/command/install.py 57.17% <100.00%> (ø)
comfy_cli/command/models/models.py 40.00% <100.00%> (ø)
comfy_cli/config_manager.py 49.38% <100.00%> (-0.62%) ⬇️
comfy_cli/file_utils.py 79.37% <100.00%> (-0.13%) ⬇️
comfy_cli/registry/config_parser.py 50.00% <ø> (-0.34%) ⬇️
comfy_cli/registry/types.py 100.00% <100.00%> (ø)
comfy_cli/standalone.py 34.52% <100.00%> (-0.78%) ⬇️
comfy_cli/typing.py 100.00% <100.00%> (ø)
... and 4 more
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@bigcat88 bigcat88 merged commit f5d6790 into main Jan 7, 2026
14 checks passed
@bigcat88 bigcat88 deleted the chore/drop-python3.9-2 branch January 7, 2026 15:32
christian-byrne added a commit to christian-byrne/comfy-cli that referenced this pull request Jan 20, 2026
PR Comfy-Org#349 changed type hints from Optional[bool] to bool | None syntax,
but typer < 0.12.4 doesn't support PEP 604 union types (X | None).
This caused RuntimeError when running comfy CLI with older typer.

Changes:
- Update typer dependency from >=0.9 to >=0.12.4
- Remove deprecated is_flag parameter from typer.Option() calls
  (is_flag was deprecated in typer 0.15.0 and never worked properly)

Fixes Comfy-Org#355

References:
- Typer PEP 604 support: fastapi/typer#548
- is_flag deprecation: fastapi/typer#986
christian-byrne added a commit to christian-byrne/comfy-cli that referenced this pull request Jan 20, 2026
PR Comfy-Org#349 changed type hints from Optional[bool] to bool | None syntax,
but typer < 0.12.4 doesn't support PEP 604 union types (X | None).
This caused RuntimeError when running comfy CLI with older typer.

Changes:
- Update typer dependency from >=0.9 to >=0.12.4
- Remove deprecated is_flag parameter from typer.Option() calls
  (is_flag was deprecated in typer 0.15.0 and never worked properly)

Fixes Comfy-Org#355

References:
- Typer PEP 604 support: fastapi/typer#548
- is_flag deprecation: fastapi/typer#986

Amp-Thread-ID: https://ampcode.com/threads/T-019bdd23-62ea-732f-9443-5705ebe27a3b
Co-authored-by: Amp <amp@ampcode.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant