[P2][ci] Faire de Makefile la source unique des contrôles locaux et CI #33

New issue

Closed

opened 2026-07-31 21:05:15 +00:00 by mathieu · 1 comment

mathieu commented

2026-07-31 21:05:15 +00:00

Owner

Constat

make test exécute seulement Go alors que CI ajoute Python (Makefile:28-39, .forgejo/workflows/ci.yml:64-111). Ni go mod tidy -diff ni govulncheck ne sont imposés. Les commandes locales documentées peuvent donc être vertes quand la CI ou les dépendances ne le sont pas.

Tâche

Créer des cibles explicites test-go, test-osd, module-check et vuln-check avec version du scanner épinglée; faire dépendre test des suites livrées et appeler les mêmes cibles depuis Forgejo.

Critères d'acceptation

make test couvre Go et le helper Python.
Une dérive go.mod/go.sum échoue sans modifier le worktree.
Un finding atteignable govulncheck échoue; une panne de base est distinguable.
README et CI n'ont plus de commandes dupliquées divergentes.

Validation

make test && make lint && go mod tidy -diff && make installcheck

## Constat `make test` exécute seulement Go alors que CI ajoute Python (`Makefile:28-39`, `.forgejo/workflows/ci.yml:64-111`). Ni `go mod tidy -diff` ni `govulncheck` ne sont imposés. Les commandes locales documentées peuvent donc être vertes quand la CI ou les dépendances ne le sont pas. ## Tâche Créer des cibles explicites `test-go`, `test-osd`, module-check et vuln-check avec version du scanner épinglée; faire dépendre `test` des suites livrées et appeler les mêmes cibles depuis Forgejo. ## Critères d'acceptation - `make test` couvre Go et le helper Python. - Une dérive go.mod/go.sum échoue sans modifier le worktree. - Un finding atteignable `govulncheck` échoue; une panne de base est distinguable. - README et CI n'ont plus de commandes dupliquées divergentes. ## Validation `make test && make lint && go mod tidy -diff && make installcheck`

hermes referenced this issue from a pull request that will close it,

2026-08-07 18:02:08 +00:00

test(ci): partager des contrôles déterministes entre Makefile et CI #46

hermes commented

2026-08-07 18:02:46 +00:00

Member

Correctif proposé dans la PR #46 : mathieu/murmur#46. Cette PR est empilée sur la PR #43 et doit être fusionnée après le bloc audio.

Correctif proposé dans la PR #46 : https://git.2027a.net/mathieu/murmur/pulls/46. Cette PR est empilée sur la PR #43 et doit être fusionnée après le bloc audio.

mathieu closed this issue

2026-09-11 19:50:41 +00:00