mirror of
https://github.com/esphome/esphome.git
synced 2026-06-24 12:51:11 +00:00
[ci] sync-device-classes: skip pylint hook
This commit is contained in:
16
.github/workflows/sync-device-classes.yml
vendored
16
.github/workflows/sync-device-classes.yml
vendored
@@ -59,8 +59,6 @@ jobs:
|
||||
- name: Install Home Assistant
|
||||
run: |
|
||||
uv pip install --system -e lib/home-assistant
|
||||
# Project requirements are needed so pylint can resolve runtime
|
||||
# imports (e.g. smpclient in esphome/components/nrf52/ota.py).
|
||||
uv pip install --system -r requirements.txt -r requirements_test.txt pre-commit
|
||||
|
||||
- name: Sync
|
||||
@@ -72,13 +70,23 @@ jobs:
|
||||
# files. pre-commit exits non-zero whenever a hook touches anything,
|
||||
# which would otherwise abort the workflow before the auto-fixes
|
||||
# can flow into the sync PR.
|
||||
#
|
||||
# pylint is skipped: this workflow only installs a subset of the
|
||||
# runtime deps (HA + requirements*.txt), so pylint surfaces
|
||||
# import-error / relative-beyond-top-level noise that the main CI
|
||||
# already gates on real PRs.
|
||||
env:
|
||||
SKIP: pylint
|
||||
run: python script/run-in-env.py pre-commit run --all-files || true
|
||||
|
||||
- name: Verify pre-commit clean
|
||||
# Second pass: re-run all hooks against the now-fixed tree.
|
||||
# Auto-fixers exit 0 (nothing to change); any remaining failure
|
||||
# from a check-only hook (pylint / flake8 / yamllint / ci-custom)
|
||||
# is a real issue and fails the workflow loudly.
|
||||
# from a check-only hook (flake8 / yamllint / ci-custom) is a
|
||||
# real issue and fails the workflow loudly. pylint stays skipped
|
||||
# for the same reason as above.
|
||||
env:
|
||||
SKIP: pylint
|
||||
run: python script/run-in-env.py pre-commit run --all-files
|
||||
|
||||
- name: Commit changes
|
||||
|
||||
Reference in New Issue
Block a user