Skip to content

ci: fix python_check — test supported HA versions, satisfy mypy - #7

Merged
Dekkee merged 3 commits into
mainfrom
ci/fix-python-check
Jun 12, 2026
Merged

ci: fix python_check — test supported HA versions, satisfy mypy#7
Dekkee merged 3 commits into
mainfrom
ci/fix-python-check

Conversation

@Dekkee

@Dekkee Dekkee commented Jun 12, 2026

Copy link
Copy Markdown
Owner

python_check failed on every commit because the matrix still tested HA 2024.2/2024.3 from upstream history, which the fork no longer supports (DataUpdateCoordinator(config_entry=...) needs 2024.8+).

  • matrix → HA 2026.5.4 + 2026.6 on Python 3.14, actions bumped off node16
  • real mypy fixes that show up against current HA: ConfigFlowResult in config_flow, HVACMode | None narrowing in climate, bool() in binary_sensor is_on, unused voluptuous ignores
  • black reformat of files that drifted

Verified locally against HA 2026.6.2: flake8 0, mypy clean (17 files), black clean.

🤖 Generated with Claude Code

Dekkee and others added 3 commits June 12, 2026 18:15
The matrix still pinned HA 2024.2/2024.3 from upstream history, which the
component no longer supports (DataUpdateCoordinator config_entry kwarg needs
2024.8+), so every commit failed CI regardless of its content. Test what the
fork actually targets: 2026.5.4 (compat branch baseline) and latest 2026.6.
Also bump checkout/setup-python actions off the deprecated node16 runtime.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
- config_flow: FlowResult → ConfigFlowResult (HA 2024.4+ flow typing)
- climate: narrow _attr_hvac_mode before returning it from hvac_mode
- binary_sensor: wrap is_on comparison in bool() (device.get returns Any)
- drop now-unused voluptuous type: ignore comments (ships typed since 0.14)

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@Dekkee
Dekkee merged commit 542c184 into main Jun 12, 2026
2 checks passed
@Dekkee
Dekkee deleted the ci/fix-python-check branch June 12, 2026 15:21
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