Running mypy via precommit with --install-type --non-interactive seem to silence some errors
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 35/100
Piste de recherche
Commencez par .pre-commit-config.yaml et reproduisez les deux commandes pre-commit mypy indiquées dans l’issue. Comparez le comportement avec et sans --non-interactive, puis vérifiez les erreurs signalées dans ipykernel/pylab/config.py, ipykernel/pylab/backend_inline.py et ipykernel/ipkernel.py. La tâche est terminée lorsque la cause des diagnostics différents est établie et que le comportement prévu de la configuration est documenté ou corrigé.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
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.
- Langage dominant
- Python
- Étoiles
- 734
- Forks
- 411
- Merge moyen
- 1 j 2 h
- PR mergées (30 j)
- 9
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de ipython/ipykernel
-
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 72/100
-
Difficulté 4/5 3-5 jours Accessibilité débutants 48/100
-
Difficulté 4/5 3-5 jours Accessibilité débutants 68/100
-
ipython/ipykernel#1550 · 1 commentaire · 1 réaction · 1 personne assignée ·
Toutes les issues de ipython/ipykernel
Issues similaires
-
essnmx good first issue
Difficulté 1/5 Moins d'une heure Accessibilité débutants 95/100
-
[Feature] 奇物选择添加优先级 Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
syfoud/Simulated_Scepter#174 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
Giskard-AI/giskard-oss#2840 · 1 commentaire ·
-
A claim comment carrying the issue number is silently declined while the workflow reports success Ouvertearea: repo bug perceived difficulty: 2
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
yeti-platform/yeti#1380 ·