Fix yamllint errors in POW configs, update CI and lint config

- Fix UART list indentation (yamllint error)
- Remove trailing spaces in display lambda
- Add missing newline at end of file
- Add ON/OFF to yamllint truthy allowed values
- Fix invalid comments rule option in .yamllint.yml
- Lint only changed files in CI (avoids blocking PRs with pre-existing issues)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
2026-02-19 11:14:51 +01:00
parent d4967c6411
commit b18ce4a2da
4 changed files with 23 additions and 16 deletions

View File

@@ -11,10 +11,18 @@ jobs:
runs-on: ubuntu-latest runs-on: ubuntu-latest
steps: steps:
- uses: actions/checkout@v4 - uses: actions/checkout@v4
- name: Get changed YAML files
id: changed
uses: tj-actions/changed-files@v46
with:
files: "**/*.yaml"
- uses: ibiqlik/action-yamllint@v3 - uses: ibiqlik/action-yamllint@v3
if: steps.changed.outputs.any_changed == 'true'
with: with:
config_file: .yamllint.yml config_file: .yamllint.yml
file_or_dir: "*.yaml common/**/*.yaml" file_or_dir: ${{ steps.changed.outputs.all_changed_files }}
validate: validate:
name: ESPHome config validation name: ESPHome config validation

View File

@@ -3,7 +3,7 @@ extends: default
rules: rules:
# ESPHome uses True/False (Python style) and yes/no # ESPHome uses True/False (Python style) and yes/no
truthy: truthy:
allowed-values: ["true", "false", "True", "False", "yes", "no"] allowed-values: ["true", "false", "True", "False", "yes", "no", "on", "off", "ON", "OFF"]
check-keys: false check-keys: false
# Lambdas and long substitutions can exceed 80 chars # Lambdas and long substitutions can exceed 80 chars
@@ -17,6 +17,5 @@ rules:
# Some files have trailing spaces; warn only # Some files have trailing spaces; warn only
trailing-spaces: enable trailing-spaces: enable
# Allow comments without a space after # in some cases (e.g. # noqa)
comments: comments:
min-spaces-before-comment: 1 require-starting-space: false

View File

@@ -20,10 +20,10 @@ packages:
common: !include common/common.yaml common: !include common/common.yaml
uart: uart:
- id: cse7766_uart - id: cse7766_uart
rx_pin: 16 rx_pin: 16
baud_rate: 4800 baud_rate: 4800
parity: EVEN parity: EVEN
binary_sensor: binary_sensor:
- platform: gpio - platform: gpio
@@ -82,7 +82,7 @@ display:
it.display_kwh(false); it.display_kwh(false);
it.printf(0, "%.1f", id(v_sensor).state); it.printf(0, "%.1f", id(v_sensor).state);
it.printf(1, "%.1f", id(a_sensor).state); it.printf(1, "%.1f", id(a_sensor).state);
} else { } else {
it.display_voltage(false); it.display_voltage(false);
it.display_kwh(true); it.display_kwh(true);
it.printf(0, "%.1f", id(wh_sensor).state); it.printf(0, "%.1f", id(wh_sensor).state);
@@ -183,4 +183,4 @@ switch:
interlock: [relay_on] interlock: [relay_on]
time: time:
- !include common/time/homeassistant.yaml - !include common/time/homeassistant.yaml

View File

@@ -20,10 +20,10 @@ packages:
common: !include common/common.yaml common: !include common/common.yaml
uart: uart:
- id: cse7766_uart - id: cse7766_uart
rx_pin: 16 rx_pin: 16
baud_rate: 4800 baud_rate: 4800
parity: EVEN parity: EVEN
binary_sensor: binary_sensor:
- platform: gpio - platform: gpio
@@ -82,7 +82,7 @@ display:
it.display_kwh(false); it.display_kwh(false);
it.printf(0, "%.1f", id(v_sensor).state); it.printf(0, "%.1f", id(v_sensor).state);
it.printf(1, "%.1f", id(a_sensor).state); it.printf(1, "%.1f", id(a_sensor).state);
} else { } else {
it.display_voltage(false); it.display_voltage(false);
it.display_kwh(true); it.display_kwh(true);
it.printf(0, "%.1f", id(wh_sensor).state); it.printf(0, "%.1f", id(wh_sensor).state);
@@ -183,4 +183,4 @@ switch:
interlock: [relay_on] interlock: [relay_on]
time: time:
- !include common/time/homeassistant.yaml - !include common/time/homeassistant.yaml