Running mypy via precommit with --install-type --non-interactive seem to silence some errors
- Linguagem predominante
- Python
- Estrelas
- 734
- Forks
- 412
- Merge médio
- 1d 5h
- PRs com merge (30d)
- 8
Descrição
```
$ 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.
Guia de contribuição
Direção de pesquisa
Start with .pre-commit-config.yaml and reproduce the two pre-commit mypy commands shown in the issue. Compare the behavior with and without --non-interactive, then check the reported errors in ipykernel/pylab/config.py, ipykernel/pylab/backend_inline.py, and ipykernel/ipkernel.py. Done means the cause of the differing diagnostics is established and the intended configuration behavior is documented or fixed.
Escrita pelo modelo de indexação a partir do texto da issue.
Avaliação
- Stack de tecnologia
- python
- Domínio
- tooling
- Tipo de issue
- Bug
- Dificuldade
- 3/5
- Tempo estimado
- 1-2 dias
- Status de atividade
- Estagnada
- Clareza
- Razoavelmente clara
- Facilidade para iniciantes
- 35/100