[Majeur] CI GitHub : les PR externes passent sans aucun test ni lint #90

New issue

Open

opened 2026-08-26 05:32:42 +00:00 by mat · 0 comments

mat commented

2026-08-26 05:32:42 +00:00

Owner

Problème

.github/workflows/validate.yml ne contient que HACS validation + hassfest. Ruff/pytest/mypy ne tournent que côté Forgejo (.forgejo/workflows/validate.yml) — c'est documenté et délibéré, mais :

le issue_tracker et la documentation du manifest pointent vers GitHub ;
un contributeur externe ouvrant une PR sur GitHub ne voit jamais les tests/lint tourner.

Recommandation

Dupliquer les 3 jobs portables (ruff / pytest / mypy) dans le workflow GitHub — ils sont « portables » par construction.
À défaut : afficher clairement dans le README que la CI de vérité est Forgejo et que les PR GitHub sont buildées après mirror.

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

## Problème `.github/workflows/validate.yml` ne contient que HACS validation + hassfest. Ruff/pytest/mypy ne tournent que côté Forgejo (`.forgejo/workflows/validate.yml`) — c'est documenté et délibéré, mais : - le `issue_tracker` et la `documentation` du manifest pointent vers **GitHub** ; - un contributeur externe ouvrant une PR sur GitHub ne voit jamais les tests/lint tourner. ## Recommandation 1. Dupliquer les 3 jobs portables (ruff / pytest / mypy) dans le workflow GitHub — ils sont « portables » par construction. 2. À défaut : afficher clairement dans le README que la CI de vérité est Forgejo et que les PR GitHub sont buildées après mirror. --- Constaté lors de la review de code pré-v1.0 (v0.8.1).

mat added the

v1.0 · majeur

label

2026-08-26 05:32:42 +00:00