ipython / ipython/ipykernel

Running mypy via precommit with --install-type --non-interactive seem to silence some errors

Ouverte
#1,336 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
Langage dominant
Python
Étoiles
734
Forks
412
Merge moyen
1 j 5 h
PR mergées (30 j)
8

Description

```
$ git diff ; pre-commit run --all-files --hook-stage manual mypy
mypy.....................................................................Passed
```
But:
```
$ git diff ; pre-commit run --all-files --hook-stage manual mypy
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
index 62514712..06f52541 100644
--- a/.pre-commit-config.yaml
+++ b/.pre-commit-config.yaml
@@ -44,7 +44,6 @@ repos:
hooks:
- id: mypy
files: ipykernel
- args: ["--install-types", "--non-interactive"]
additional_dependencies:
[
"traitlets>=5.13",
mypy.....................................................................Failed
- hook id: mypy
- exit code: 1

ipykernel/pylab/config.py:8: error: Unused "type: ignore" comment
[unused-ignore]
from matplotlib_inline.config import * # type:ignore[import-untyped] ...
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~...
ipykernel/pylab/backend_inline.py:8: error: Unused "type: ignore" comment
[unused-ignore]
from matplotlib_inline.backend_inline import * # type:ignore[import-u...
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~...
ipykernel/ipkernel.py:162: error: Unused "type: ignore" comment
[unused-ignore]
import appnope # type:ignore[import-untyped]
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Found 3 errors in 3 files (checked 48 source files)
```

It seem to be just `--install-types`

```
$ git diff ; pre-commit run --all-files --hook-stage manual mypy
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
index 62514712..e56cd12c 100644
--- a/.pre-commit-config.yaml
+++ b/.pre-commit-config.yaml
@@ -44,7 +44,7 @@ repos:
hooks:
- id: mypy
files: ipykernel
- args: ["--install-types", "--non-interactive"]
+ args: ["--install-types"]
additional_dependencies:
[
"traitlets>=5.13",
mypy.....................................................................Passed
```

I also get way more errors running mypy manually, but I think this is because of missing dependencies.

Guide de contribution

Ouvrir le guide de contribution

Évaluation

Cette issue n'a pas encore été évaluée.

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.