Skip to content
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.

Commit b05963c

Browse files
committedMar 29, 2024
Expand and clarify test module docstring
About why there are so many separate mypy suppressions even when they could be consolidated into a smaller number in some places.
1 parent 247dc15 commit b05963c

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed
 

‎test/deprecation/test_attributes.py

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,11 @@
11
"""Tests for dynamic and static attribute errors in GitPython's top-level git module.
22
33
Provided mypy has ``warn_unused_ignores = true`` set, running mypy on these test cases
4-
checks static typing of the code under test. (Running pytest checks dynamic behavior.)
4+
checks static typing of the code under test. This is the reason for the many separate
5+
single-line attr-defined suppressions, so those should not be replaced with a smaller
6+
number of more broadly scoped suppressions, even where it is feasible to do so.
7+
8+
Running pytest checks dynamic behavior as usual.
59
"""
610

711
from itertools import groupby

0 commit comments

Comments
 (0)
Please sign in to comment.