[Bloquant v1.0] Le job CI mypy ne vérifie rien (`ignore_errors = True` sur tout le package) #83

New issue

Open

opened 2026-08-26 05:31:54 +00:00 by mat · 0 comments

mat commented

2026-08-26 05:31:54 +00:00

Owner

Problème

mypy.ini:20-21 :

[mypy-custom_components.comwatt.*]
ignore_errors = True

Le job type-check-mypy de .forgejo/workflows/validate.yml passe donc toujours, quel que soit le code (« Success: no issues found in 8 source files » est un faux signal).

Le commentaire du fichier dit que l'override est temporaire et couvre 2 findings ciblés (« C1 et sensor._device ») — mais il est appliqué à tout le package.

Recommandation

Réduire l'override aux fichiers/fonctions réellement concernés ([mypy-custom_components.comwatt.sensor] etc.), corriger les 2 findings si possible, et faire échouer le job sur de vraies erreurs.
À défaut de temps : supprimer le job mypy de la CI plutôt que d'afficher un feu vert mensonger.
Vérifier ce que mypy détecte réellement une fois l'override retiré : mypy local avec ignore_errors = False.

Constaté lors de la review de code pré-v1.0 (v0.8.1).

## Problème `mypy.ini:20-21` : ```ini [mypy-custom_components.comwatt.*] ignore_errors = True ``` Le job `type-check-mypy` de `.forgejo/workflows/validate.yml` passe donc **toujours**, quel que soit le code (« Success: no issues found in 8 source files » est un faux signal). Le commentaire du fichier dit que l'override est temporaire et couvre 2 findings ciblés (« C1 et sensor._device ») — mais il est appliqué à tout le package. ## Recommandation 1. Réduire l'override aux fichiers/fonctions réellement concernés (`[mypy-custom_components.comwatt.sensor]` etc.), corriger les 2 findings si possible, et faire échouer le job sur de vraies erreurs. 2. À défaut de temps : supprimer le job mypy de la CI plutôt que d'afficher un feu vert mensonger. 3. Vérifier ce que mypy détecte réellement une fois l'override retiré : `mypy` local avec `ignore_errors = False`. --- Constaté lors de la review de code pré-v1.0 (v0.8.1).

mat added the

v1.0 · bloquant

label

2026-08-26 05:31:54 +00:00