Skip to content

Sync review fixes from ha-core and relocate generate_ja#31

Merged
sayurin merged 1 commit into
masterfrom
refactor
May 16, 2026
Merged

Sync review fixes from ha-core and relocate generate_ja#31
sayurin merged 1 commit into
masterfrom
refactor

Conversation

@sayurin

@sayurin sayurin commented May 16, 2026

Copy link
Copy Markdown
Owner

Mirror the runtime fixes applied to the upstream homeassistant/components/echonet_lite integration (drop assert-based properties on _RuntimeController, guarantee protocol.close() in the config-flow multicast probe, replace del hass with # noqa: ARG001 in diagnostics, document the byte_offset > 0 filter, remove redundant _attr_has_entity_name declarations on platform entities, refresh quality_scale.yaml comments).

Also move the build-only generator out of the integration package so HACS users no longer receive dead code:

  • Remove generate_strings.py and strings_static.json (they only target the ha-core repo layout).
  • Rename generator/ to a top-level scripts/ directory.
  • Update generate_ja.py to absolute imports via a sys.path shim; invoke with python scripts/generate_ja.py.

Mirror the runtime fixes applied to the upstream
`homeassistant/components/echonet_lite` integration (drop
assert-based properties on `_RuntimeController`, guarantee
`protocol.close()` in the config-flow multicast probe, replace
`del hass` with `# noqa: ARG001` in diagnostics, document the
`byte_offset > 0` filter, remove redundant `_attr_has_entity_name`
declarations on platform entities, refresh `quality_scale.yaml`
comments).

Also move the build-only generator out of the integration package so
HACS users no longer receive dead code:

- Remove `generate_strings.py` and `strings_static.json` (they only
  target the ha-core repo layout).
- Rename `generator/` to a top-level `scripts/` directory.
- Update `generate_ja.py` to absolute imports via a `sys.path` shim;
  invoke with `python scripts/generate_ja.py`.
@sayurin sayurin merged commit 33784aa into master May 16, 2026
2 checks passed
@sayurin sayurin deleted the refactor branch May 16, 2026 09:14
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