Fix yamllint (#6253)

2024-11-23 23:48:11 +01:00 · 2024-02-21 17:14:30 +13:00 · 2024-02-21 17:14:30 +13:00 · 75af4c3d62
commit 75af4c3d62
parent e847039ffd
23 changed files with 83 additions and 70 deletions
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@ -11,6 +11,7 @@ on:
      - "**"
      - "!.github/workflows/*.yml"
      - ".github/workflows/ci.yml"
+      - "!.yamllint"
  merge_group:

 permissions:
--- a/.github/workflows/needs-docs.yml
+++ b/.github/workflows/needs-docs.yml
@ -1,5 +1,6 @@
 name: Needs Docs

+# yamllint disable-line rule:truthy
 on:
  pull_request:
    types: [labeled, unlabeled]
--- a/.github/workflows/sync-device-classes.yml
+++ b/.github/workflows/sync-device-classes.yml
@ -1,6 +1,7 @@
 ---
 name: Synchronise Device Classes from Home Assistant

+# yamllint disable-line rule:truthy
 on:
  workflow_dispatch:
  schedule:
--- a/.github/workflows/yaml-lint.yml
+++ b/.github/workflows/yaml-lint.yml
@ -1,5 +1,7 @@
+---
 name: YAML lint

+# yamllint disable-line rule:truthy
 on:
  push:
    branches: [dev, beta, release]
@ -20,3 +22,5 @@ jobs:
        uses: actions/checkout@v4.1.1
      - name: Run yamllint
        uses: frenck/action-yamllint@v1.5.0
+        with:
+          strict: true
--- a/.yamllint
+++ b/.yamllint
@ -1,3 +1,18 @@
 ---
-ignore: |
-  venv/
+extends: default
+
+ignore-from-file: .gitignore
+
+rules:
+  document-start: disable
+  empty-lines:
+    level: error
+    max: 1
+    max-start: 0
+    max-end: 1
+  indentation:
+    level: error
+    spaces: 2
+    indent-sequences: true
+    check-multi-line-strings: false
+  line-length: disable
--- a/tests/components/animation/test.esp32-c3-idf.yaml
+++ b/tests/components/animation/test.esp32-c3-idf.yaml
@ -20,4 +20,4 @@ animation:
  - id: rgb565_animation
    file: ../../pnglogo.png
    type: RGB565
-    use_transparency: no
+    use_transparency: false
--- a/tests/components/animation/test.esp32-c3.yaml
+++ b/tests/components/animation/test.esp32-c3.yaml
@ -20,4 +20,4 @@ animation:
  - id: rgb565_animation
    file: ../../pnglogo.png
    type: RGB565
-    use_transparency: no
+    use_transparency: false
--- a/tests/components/animation/test.esp32-idf.yaml
+++ b/tests/components/animation/test.esp32-idf.yaml
@ -20,4 +20,4 @@ animation:
  - id: rgb565_animation
    file: ../../pnglogo.png
    type: RGB565
-    use_transparency: no
+    use_transparency: false
--- a/tests/components/animation/test.esp32.yaml
+++ b/tests/components/animation/test.esp32.yaml
@ -20,4 +20,4 @@ animation:
  - id: rgb565_animation
    file: ../../pnglogo.png
    type: RGB565
-    use_transparency: no
+    use_transparency: false
--- a/tests/components/animation/test.esp8266.yaml
+++ b/tests/components/animation/test.esp8266.yaml
@ -20,4 +20,4 @@ animation:
  - id: rgb565_animation
    file: ../../pnglogo.png
    type: RGB565
-    use_transparency: no
+    use_transparency: false
--- a/tests/components/animation/test.rp2040.yaml
+++ b/tests/components/animation/test.rp2040.yaml
@ -20,4 +20,4 @@ animation:
  - id: rgb565_animation
    file: ../../pnglogo.png
    type: RGB565
-    use_transparency: no
+    use_transparency: false
--- a/tests/components/mopeka_std_check/test.esp32.yaml
+++ b/tests/components/mopeka_std_check/test.esp32.yaml
@ -13,4 +13,3 @@ sensor:
      name: "Propane test distance"
    battery_level:
      name: "Propane test battery level"
-
--- a/tests/test1.yaml
+++ b/tests/test1.yaml
@ -380,7 +380,7 @@ ble_client:
      then:
        - ble_client.numeric_comparison_reply:
            id: ble_blah
-            accept: True
+            accept: true
  - mac_address: C4:4F:33:11:22:33
    id: my_bedjet_ble_client

@ -2053,7 +2053,7 @@ binary_sensor:
    on_press:
      - fan.cycle_speed:
          id: fan_speed
-          off_speed_cycle: False
+          off_speed_cycle: false
      - logger.log: "Cycle speed clicked"
  - platform: remote_receiver
    name: Raw Remote Receiver Test
@ -3411,7 +3411,7 @@ display:
    reset_pin:
      allow_other_uses: true
      number: GPIO23
-    backlight_pin: no
+    backlight_pin: false
    lambda: |-
      it.rectangle(0, 0, it.get_width(), it.get_height());
  - platform: st7920
--- a/tests/test2.yaml
+++ b/tests/test2.yaml
@ -8,7 +8,7 @@ esphome:
 globals:
  - id: my_global_string
    type: std::string
-    restore_value: yes
+    restore_value: true
    max_restore_data_length: 70
    initial_value: '"DefaultValue"'

@ -786,11 +786,11 @@ image:
  - id: rgb24_image
    file: pnglogo.png
    type: RGB24
-    use_transparency: yes
+    use_transparency: true
  - id: rgb565_image
    file: pnglogo.png
    type: RGB565
-    use_transparency: no
+    use_transparency: false
  - id: web_svg_image
    file: https://raw.githubusercontent.com/esphome/esphome-docs/a62d7ab193c1a464ed791670170c7d518189109b/images/logo.svg
    resize: 256x48
--- a/tests/test3.1.yaml
+++ b/tests/test3.1.yaml
@ -503,7 +503,7 @@ switch:
  - platform: template
    name: open_vent
    id: open_vent
-    optimistic: True
+    optimistic: true
    on_turn_on:
      then:
        - grove_tb6612fng.run:
--- a/tests/test3.yaml
+++ b/tests/test3.yaml
@ -1035,14 +1035,14 @@ climate:
        target_temperature: 1
        current_temperature: 0.5
    supported_modes:
-    - 'OFF'
+      - "OFF"
      - HEAT_COOL
      - COOL
      - HEAT
      - DRY
      - FAN_ONLY
    supported_swing_modes:
-    - 'OFF'
+      - "OFF"
      - VERTICAL
      - HORIZONTAL
      - BOTH
@ -1055,13 +1055,13 @@ climate:
      then:
        - logger.log:
            level: DEBUG
-            format: "Alarm activated. Code: %d. Message: \"%s\""
+            format: 'Alarm activated. Code: %d. Message: "%s"'
            args: [code, message]
    on_alarm_end:
      then:
        - logger.log:
            level: DEBUG
-            format: "Alarm deactivated. Code: %d. Message: \"%s\""
+            format: 'Alarm deactivated. Code: %d. Message: "%s"'
            args: [code, message]

 sprinkler:
--- a/tests/test4.yaml
+++ b/tests/test4.yaml
@ -477,7 +477,6 @@ binary_sensor:
      sx1509: sx1509_hub
      number: 3

-
  - platform: touchscreen
    touchscreen_id: lilygo_touchscreen
    id: touch_key1
@ -491,7 +490,6 @@ binary_sensor:
    id: touch_key_911
    index: 0

-
  - platform: gpio
    name: MaxIn Pin 4
    pin:
@ -521,7 +519,6 @@ binary_sensor:
        input: true
      inverted: false

-
 climate:
  - platform: tuya
    id: tuya_climate
@ -591,7 +588,6 @@ cover:
    open_duration: 14s
    close_duration: 14s

-
 display:
  - platform: addressable_light
    id: led_matrix_32x8_display
@ -1004,7 +1000,6 @@ touchscreen:
      number: GPIO3
    display: inkplate_display

-
  - platform: ft63x6
    id: ft63_touchscreen
    interrupt_pin:
--- a/tests/test6.yaml
+++ b/tests/test6.yaml
@ -42,7 +42,6 @@ switch:
    output: pin_4
    id: pin_4_switch

-
 spi:  # Pins are for SPI1 on the RP2040 Pico-W
  miso_pin: 8
  clk_pin: 10
@ -69,7 +68,6 @@ spi:  # Pins are for SPI1 on the RP2040 Pico-W
 #    bit1_high: .69us
 #    bit1_low: .4us

-
 sensor:
  - platform: internal_temperature
    name: Internal Temperature
--- a/tests/test8.1.yaml
+++ b/tests/test8.1.yaml
@ -31,8 +31,7 @@ spi:
  - id: quad_spi
    clk_pin: 47
    data_pins:
-      -
-        number: 40
+      - number: 40
        allow_other_uses: false
      - 41
      - 42
--- a/tests/test8.yaml
+++ b/tests/test8.yaml
@ -101,4 +101,4 @@ animation:
  - id: rgb565_animation
    file: pnglogo.png
    type: RGB565
-    use_transparency: no
+    use_transparency: false