Skip to content

Préfixe les cibles Makefile à lancer dans le container par docker: - #2393

Open
Korbeil wants to merge 1 commit into
afup:masterfrom
Korbeil:afup-identify-make-actions-that
Open

Korbeil wants to merge 1 commit into
afup:masterfrom
Korbeil:afup-identify-make-actions-that

Conversation

@Korbeil

@Korbeil Korbeil commented Sep 14, 2026 •

Copy link
Copy Markdown
Member

Description

Les cibles Makefile à lancer dans le container sont maintenant préfixées par docker: (tests, qualité, dépendances, assets, base de données). Celles à lancer depuis la machine, qui orchestrent docker, n'ont pas de préfixe (make up, make init, make console…). Attention : certaines commandes changent de nom (make test devient make docker:test dans le container, par exemple), il faut mettre à jour ses habitudes et relancer make hooks.

Changements

  • Les cibles de tests et de qualité (docker:test, docker:phpstan, docker:cs-fix, docker:rector…) s'exécutent directement dans le container, ce qui corrige le hook pre-commit
  • make install passe par le container au lieu d'exécuter composer et npm sur la machine
  • La convention est documentée dans l'en-tête du Makefile et dans la doc de dev ; la CI et les hooks git utilisent les nouveaux noms

Comment tester

  1. make help : la liste montre les cibles locales sans préfixe et les cibles container en docker:*
  2. make up && make init depuis la machine : les containers démarrent et l'initialisation passe par le container
  3. make console puis make docker:test : les tests unitaires et PHP CS Fixer se lancent dans le container

@Korbeil
Korbeil force-pushed the afup-identify-make-actions-that branch 2 times, most recently from 248c602 to 13620a5 Compare September 14, 2026 13:06
@Korbeil
Korbeil marked this pull request as ready for review September 14, 2026 13:10
@Korbeil
Korbeil force-pushed the afup-identify-make-actions-that branch from 13620a5 to bd6c253 Compare September 14, 2026 13:20
@Korbeil
Korbeil force-pushed the afup-identify-make-actions-that branch 2 times, most recently from 75063c0 to fa2e28d Compare October 5, 2026 19:33
@Korbeil
Korbeil force-pushed the afup-identify-make-actions-that branch from fa2e28d to 3910c47 Compare October 21, 2026 11:44
@Korbeil Korbeil changed the title Identify make actions and prefix them with docker-*/local-* Préfixe les cibles Makefile à lancer dans le container par docker: Oct 21, 2026
@Korbeil
Korbeil force-pushed the afup-identify-make-actions-that branch from 3910c47 to 17f450c Compare October 25, 2026 13:00
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.

1 participant