[Majeur] Ignores ruff « temporaires » obsolètes (E501/E711/F401 à 0 violations, 6 I001 auto-fixables) #95

Open
opened 2026-08-26 05:32:44 +00:00 by mat · 0 comments
Owner

Problème

pyproject.toml:18-23 ignore E501/E711/F401/I001 avec un commentaire « Temporary ignores — each is tracked by a finding ».

Mesuré (ruff 0.15.x) :

  • 0 violation E501, E711, F401 → les ignores sont devenus faux.
  • 6 × I001 (imports non triés) : __init__.py, config_flow.py, coordinator.py, test_config_flow.py, test_coordinator.py, test_init.py — toutes auto-fixables par ruff check --fix (imports purs, aucun conflit à prévoir).

Recommandation

  1. ruff check --fix puis retirer les 4 ignores de pyproject.toml.
  2. 15 minutes de travail pour verrouiller la rigueur avant v1.0 — sinon la dette revient.

Bonus de cohérence repéré au passage : mypy.ini dit python_version = 3.13 vs pyproject.toml target-version = "py312" — harmoniser.


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

## Problème `pyproject.toml:18-23` ignore E501/E711/F401/I001 avec un commentaire « Temporary ignores — each is tracked by a finding ». Mesuré (ruff 0.15.x) : - **0 violation** E501, E711, F401 → les ignores sont devenus faux. - **6 × I001** (imports non triés) : `__init__.py`, `config_flow.py`, `coordinator.py`, `test_config_flow.py`, `test_coordinator.py`, `test_init.py` — toutes auto-fixables par `ruff check --fix` (imports purs, aucun conflit à prévoir). ## Recommandation 1. `ruff check --fix` puis retirer les 4 ignores de `pyproject.toml`. 2. 15 minutes de travail pour verrouiller la rigueur avant v1.0 — sinon la dette revient. Bonus de cohérence repéré au passage : `mypy.ini` dit `python_version = 3.13` vs `pyproject.toml` `target-version = "py312"` — harmoniser. --- Constaté lors de la review de code pré-v1.0 (v0.8.1).
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
mat/homeassistant-comwatt#95
No description provided.