Skip to content

Commit fa0b849

Browse files
nickwinderclaude
andcommitted
Fix ruff per-file-ignores and remove unused variable in test
- pyproject.toml: add D102 to tests/* per-file-ignores so test methods are not required to have docstrings (the comment indicated this intent but the empty array had no effect) - test_client.py: remove unused `result` assignment in test_password_protect_pdf_with_permissions Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent c92f342 commit fa0b849

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

pyproject.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -112,7 +112,7 @@ ignore = [
112112
convention = "google"
113113

114114
[tool.ruff.lint.per-file-ignores]
115-
"tests/*" = [] # Don't require docstrings in tests, allow asserts
115+
"tests/*" = ["D102"] # Don't require docstrings in tests
116116

117117
[tool.mypy]
118118
python_version = "3.10"

tests/unit/test_client.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -737,7 +737,7 @@ async def test_password_protect_pdf_with_permissions(
737737
owner_password = "owner456"
738738
permissions = ["printing", "extract_accessibility"]
739739

740-
result = await unit_client.password_protect(
740+
await unit_client.password_protect(
741741
file, user_password, owner_password, permissions
742742
)
743743

0 commit comments

Comments
 (0)