Skip to content

fix: support variable-length channel args in make visualizer - #280

Merged
raveriss merged 1 commit into
mainfrom
codex/fix-makefile-to-handle-channels-argument
Feb 5, 2026
Merged

raveriss merged 1 commit into
mainfrom
codex/fix-makefile-to-handle-channels-argument

Conversation

@raveriss

@raveriss raveriss commented Feb 5, 2026

Copy link
Copy Markdown
Owner

Motivation

  • Corriger la cible visualizer du Makefile qui rejetait --channels (GNU Make interprète --* comme option) et ne supportait pas un nombre variable d'arguments pour les canaux.

Description

  • Parse des goals additionnels via wordlist pour récupérer tous les arguments après <subject> <run> et construire dynamiquement l'option --channels si nécessaire.
  • Normalisation d'un éventuel préfixe --channels (pour le cas make -- visualizer ...) afin que la cible accepte à la fois make visualizer 1 3 C3 Cz C4 et make -- visualizer 1 3 --channels C3 Cz C4.
  • Fichier modifié : Makefile (cible visualizer).

Testing

  • Exécution de la suite de tests unitaires via poetry run pytest -q (577 tests), tous passés.
  • Exécution de la séquence de contrôle statique pre-commit / black --check / isort --check-only / ruff / mypy / pip-audit : toutes les vérifications automatiques sont passées après mise à jour temporaire de pip pour pip-audit.
  • Couverture et gate CI locale via make cov ont produit un rapport et ont atteint la couverture totale requise (90% global), avec génération de coverage.json et coverage.xml réussie.
  • Vérification fonctionnelle de la cible Make en mode dry-run a confirmé la construction correcte de la commande pour les deux syntaxes testées : make -n visualizer 1 3 C3 Cz C4 et make -n -- visualizer 1 3 --channels C3 Cz C4, toutes deux valides.

Codex Task

@codecov

codecov Bot commented Feb 5, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@raveriss
raveriss marked this pull request as ready for review February 5, 2026 20:23
@raveriss
raveriss merged commit 7dc7f54 into main Feb 5, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant