Remove stale ty: ignore suppression comments #60

Closed
opened 2026-06-30 03:59:33 +00:00 by coding-agent-marvin8 · 1 comment
coding-agent-marvin8 commented 2026-06-30 03:59:33 +00:00 (Migrated from codeberg.org)

ty check now resolves three imports/types that previously required suppression comments. The stale directives cause ty to exit with an error.

Affected files:

  • packages/fenliu/hooks/llms_txt.py:7-8 — two stale # ty: ignore[unresolved-import] on mkdocs imports
  • packages/zhongli/tests/test_attachment_hasher.py:44 — stale # type: ignore on AsyncMock assignment
ty check now resolves three imports/types that previously required suppression comments. The stale directives cause ty to exit with an error. Affected files: - packages/fenliu/hooks/llms_txt.py:7-8 — two stale `# ty: ignore[unresolved-import]` on mkdocs imports - packages/zhongli/tests/test_attachment_hasher.py:44 — stale `# type: ignore` on AsyncMock assignment
coding-agent-marvin8 commented 2026-06-30 03:59:40 +00:00 (Migrated from codeberg.org)

Plan: remove the three suppression comments, verify ty check passes, one commit per package or one combined commit.

Files to touch:

  1. packages/fenliu/hooks/llms_txt.py — delete trailing # ty: ignore[unresolved-import] on lines 7–8
  2. packages/zhongli/tests/test_attachment_hasher.py — delete trailing # type: ignore on line 44
Plan: remove the three suppression comments, verify ty check passes, one commit per package or one combined commit. Files to touch: 1. packages/fenliu/hooks/llms_txt.py — delete trailing `# ty: ignore[unresolved-import]` on lines 7–8 2. packages/zhongli/tests/test_attachment_hasher.py — delete trailing `# type: ignore` on line 44
Sign in to join this conversation.
No labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
marvin8/dujiangyan#60
No description provided.