danvics/soulsync
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

Lint: silence S110 on two intentional best-effort swallows in video handlers (pre-existing)

Browse source 
ruff S110 flagged two try/except/pass in video handlers that predate this work.
Both are deliberate (a progress-log failure must not abort pruning; a probe's
uncertainty just keeps probing) — extend the existing BLE001 noqa to S110 with
the rationale. ruff check . is clean again.
This commit is contained in:
BoulderBadgeDad 2026-06-21 23:35:05 -07:00
parent f111b5dda6
commit 1b39e6aa3f
2 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
core/automation/handlers/video_auto_wishlist_airing.py
View file

@ -108,7 +108,7 @@ def prune_ended_show_follows(deps, automation_id=None, *, fetch_follows=None,
deps.update_progress( deps.update_progress(
automation_id, log_line="Removed ended show '%s' from the watchlist" automation_id, log_line="Removed ended show '%s' from the watchlist"
% (f.get('title') or tid), log_type='info') % (f.get('title') or tid), log_type='info')
except Exception: # noqa: BLE001 except Exception: # noqa: BLE001, S110 - a progress-log failure must not abort pruning
pass pass
return removed return removed

2
core/automation/handlers/video_scan_library.py
View file

@ -253,7 +253,7 @@ def probe_present_libraries(candidates, has_item, sleep, *, grace_seconds: int =
if has_item(scope, item): if has_item(scope, item):
present.add(scope) present.add(scope)
continue continue
except Exception: # noqa: BLE001 - uncertainty → keep probing, then scan except Exception: # noqa: BLE001, S110 - uncertainty → keep probing, then scan
pass pass
still.append((scope, item)) still.append((scope, item))
pending = still pending = still