Skip to content

Mettre le code en conformité avec pylint - #41

Merged
fwininger merged 1 commit into
mainfrom
fix/pylint-conformite
Sep 7, 2026
Merged

Mettre le code en conformité avec pylint#41
fwininger merged 1 commit into
mainfrom
fix/pylint-conformite

Conversation

@fwininger

Copy link
Copy Markdown
Member

.pylintrc exige fail-under=10.0 avec disable= vide, mais le dépôt plafonnait à 9.38/10 avec pylint 4 : la CI lint était rouge indépendamment des changements proposés.

Configuration :

  • retire suggestion-mode, supprimé dans pylint 4 (E0015)
  • qualifie overgeneral-exceptions en builtins.*, seule forme acceptée désormais ; cela supprime deux UserWarning et réactive la détection des except trop larges

Code :

  • _compute_single_target délègue à Executor, qui contenait déjà exactement la même logique de chargement de resource sans être utilisé (R0801, C0415)
  • merge passe par get_technologies / get_technology_data au lieu de lire other._data (W0212)
  • git_resource importe tempfile et shutil en tête (C0415) et initialise _tmp_dir dans __init__, ce qui remplace le garde hasattr (W0201)
  • compute_hashes conserve la variadique de Resource.compute_hashes dans les deux implémentations (W0221)
  • retire as_completed, jamais utilisé (W0611)
  • le except Exception de la boucle de résultats est volontaire (un dépôt en échec ne doit pas interrompre les autres) : disable local et commenté

Tests :

  • docstrings manquantes, open() sans encoding, comparaisons à {} / [], import local et import inutilisé

pylint passe à 10.00/10 sans warning, 75 tests au vert.

`.pylintrc` exige `fail-under=10.0` avec `disable=` vide, mais le dépôt
plafonnait à 9.38/10 avec pylint 4 : la CI lint était rouge indépendamment
des changements proposés.

Configuration :
- retire `suggestion-mode`, supprimé dans pylint 4 (E0015)
- qualifie `overgeneral-exceptions` en `builtins.*`, seule forme acceptée
  désormais ; cela supprime deux UserWarning et réactive la détection des
  `except` trop larges

Code :
- `_compute_single_target` délègue à `Executor`, qui contenait déjà exactement
  la même logique de chargement de resource sans être utilisé (R0801, C0415)
- `merge` passe par `get_technologies` / `get_technology_data` au lieu de lire
  `other._data` (W0212)
- `git_resource` importe `tempfile` et `shutil` en tête (C0415) et initialise
  `_tmp_dir` dans `__init__`, ce qui remplace le garde `hasattr` (W0201)
- `compute_hashes` conserve la variadique de `Resource.compute_hashes` dans
  les deux implémentations (W0221)
- retire `as_completed`, jamais utilisé (W0611)
- le `except Exception` de la boucle de résultats est volontaire (un dépôt en
  échec ne doit pas interrompre les autres) : disable local et commenté

Tests :
- docstrings manquantes, `open()` sans `encoding`, comparaisons à `{}` / `[]`,
  import local et import inutilisé

pylint passe à 10.00/10 sans warning, 75 tests au vert.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@fwininger
fwininger merged commit a1cbc6c into main Sep 7, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants