Skip to content

Type ignore comments erroneously marked as unused by dmypy #15043

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 14 commits into from
Jun 21, 2025
Merged

Type ignore comments erroneously marked as unused by dmypy #15043

Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
1ce2f21
Add test to demonstrate issue #9655
meshy Mar 4, 2023
7d9c06a
Prove ignores without error codes also disappear
meshy Mar 12, 2023
9cd625d
Prove possibly-undefined errors disappear
meshy Mar 12, 2023
94e5df4
Test run-running dmypy after a file is altered
meshy Mar 31, 2023
b5cc83f
Add failing tests showing disappearing errors
meshy Apr 13, 2023
99eda13
Fix disappearing errors when re-running dmypy check
meshy Jul 20, 2023
d798c5a
Add a regression test for type ignores in dmypy
meshy Apr 12, 2023
ad94241
Add regression test for attrs plugin
seddonym Jan 2, 2024
22777be
Don't remove modules with descendants
meshy Jan 2, 2024
64cbe6b
Remove unrequired changes to reprocess_nodes
meshy Jan 11, 2024
6e34703
Merge branch 'master' into unexpected-unused-type-ignore
hauntsaninja Apr 2, 2025
6c378b1
Merge branch 'master' into unexpected-unused-type-ignore
ilevkivskyi Jun 20, 2025
d58936a
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] Jun 20, 2025
6dd9c2e
Delete part that is likely not needed anymore
ilevkivskyi Jun 20, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Prove ignores without error codes also disappear 
This test fails with the error:

    AssertionError: Command 3 (dmypy check -- bar.py) did not give expected output
    --- Captured stderr call ---
    Expected:
      bar.py:2: error: "type: ignore" comment without error code  [ignore-without-code] (diff)
      == Return code: 1                             (diff)
    Actual:
      (empty)

This test illustrates that '"type: ignore" comment without error code'
errors currently disappear in the same way that 'Unused "type: ignore"'
errors do as described in #9655.

Ref: #9655
  • Loading branch information
@meshy
meshy committed Jan 2, 2024
commit 7d9c06a108d01903df6acca238eef1f0c97cf90c
17 changes: 17 additions & 0 deletions test-data/unit/daemon.test
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -632,3 +632,20 @@ bar.py:2: error: Unused "type: ignore" comment
[file bar.py]
from foo.empty import *
a = 1 # type: ignore

[case testTypeIgnoreWithoutCodePreservedOnRerun]
-- Regression test for https://github.com/python/mypy/issues/9655
$ dmypy start -- --enable-error-code ignore-without-code --no-error-summary
Daemon started
$ dmypy check -- bar.py
bar.py:2: error: "type: ignore" comment without error code [ignore-without-code]
== Return code: 1
$ dmypy check -- bar.py
bar.py:2: error: "type: ignore" comment without error code [ignore-without-code]
== Return code: 1

[file foo/__init__.py]
[file foo/empty.py]
[file bar.py]
from foo.empty import *
a = 1 # type: ignore