Skip to content

Commit a242e19

Browse files
address review nits: pyrefly comment wording + empty-except docs
- pyproject.toml: drop stale baseline wording; state zero-error CI policy directly (coderabbit). - test_request_body_extraction: annotate `except ImportError: pass` as skipping adapters whose optional deps aren't installed (github-code- quality empty-except flag). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent 66086f6 commit a242e19

2 files changed

Lines changed: 4 additions & 5 deletions

File tree

pyproject.toml

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -121,5 +121,4 @@ replace-imports-with-any = [
121121
]
122122

123123
[tool.pyrefly.errors]
124-
# Default severity. Known issues tracked in .pyrefly-baseline.json —
125-
# new errors fail CI, existing ones are allowed.
124+
# CI policy: zero pyrefly errors.

tests/test_request_body_extraction.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -135,23 +135,23 @@ def _adapters() -> list[tuple[str, Any]]:
135135

136136
result.append(("github", GitHubAdapter._get_request_body))
137137
except ImportError:
138-
pass
138+
pass # Optional platform dep missing — skip this adapter's cases.
139139

140140
# Telegram (static method)
141141
try:
142142
from chat_sdk.adapters.telegram.adapter import TelegramAdapter
143143

144144
result.append(("telegram", TelegramAdapter._get_request_body))
145145
except ImportError:
146-
pass
146+
pass # Optional platform dep missing — skip this adapter's cases.
147147

148148
# WhatsApp (static method)
149149
try:
150150
from chat_sdk.adapters.whatsapp.adapter import WhatsAppAdapter
151151

152152
result.append(("whatsapp", WhatsAppAdapter._get_request_body))
153153
except ImportError:
154-
pass
154+
pass # Optional platform dep missing — skip this adapter's cases.
155155

156156
return result
157157

0 commit comments

Comments
 (0)