refactor(core): src/app structure + fix auto-merge label gate #10

Merged
Corentin merged 2 commits from feat/p2-core-restructure into dev 2026-06-15 17:01:11 +02:00
Owner

Deux corrections, suite a un bug auto-merge :

  1. refactor(core) : deplacement du code applicatif src/{Core,Controllers,Views} -> src/app/{...}. Le prefixe PSR-4 App\ pointe desormais sur src/app/ (namespace miroir du dossier, separe des docroots web src/public/). Chemins ajustes (front controller + bootstrap test), commentaires alignes. Aucune modif Docker/vhost.

  2. fix(ci) : le job auto-merge filtrait le label via if: contains(github.event.pull_request.labels.*.name, ...) — expression NON fiable sur Forgejo (elle matchait meme sans label, ce qui a auto-merge PR #9 sans label). Le gate se fait maintenant en shell via lAPI (GET issues/PR/labels + grep). if reduit a event == pull_request.

Verifie : endpoint live /api/health -> categories:9, / -> 200 ; PHPUnit 23 tests OK ; PHPStan niveau 6 -> 0 erreur ; YAML valide.

NB : dev avait recu le Core en src/Core (ancienne structure) via le merge accidentel de PR #9 ; cette PR corrige la structure ET le gate. Sans label -> reste ouverte pour relecture (le fix se protege lui-meme).

Base : dev. Bloc 2 / Bloc 5 (CI).

Deux corrections, suite a un bug auto-merge : 1. refactor(core) : deplacement du code applicatif src/{Core,Controllers,Views} -> src/app/{...}. Le prefixe PSR-4 App\ pointe desormais sur src/app/ (namespace miroir du dossier, separe des docroots web src/public/). Chemins ajustes (front controller + bootstrap test), commentaires alignes. Aucune modif Docker/vhost. 2. fix(ci) : le job auto-merge filtrait le label via if: contains(github.event.pull_request.labels.*.name, ...) — expression NON fiable sur Forgejo (elle matchait meme sans label, ce qui a auto-merge PR #9 sans label). Le gate se fait maintenant en shell via lAPI (GET issues/PR/labels + grep). if reduit a event == pull_request. Verifie : endpoint live /api/health -> categories:9, / -> 200 ; PHPUnit 23 tests OK ; PHPStan niveau 6 -> 0 erreur ; YAML valide. NB : dev avait recu le Core en src/Core (ancienne structure) via le merge accidentel de PR #9 ; cette PR corrige la structure ET le gate. Sans label -> reste ouverte pour relecture (le fix se protege lui-meme). Base : dev. Bloc 2 / Bloc 5 (CI).
Corentin added 2 commits 2026-06-15 16:36:51 +02:00
fix(ci): gate auto-merge by label via API shell check (Forgejo if-expression matched even unlabeled)
All checks were successful
CI / static-tests (pull_request) Successful in 34s
CI / auto-merge (push) Has been skipped
CI / auto-merge (pull_request) Successful in 4s
CI / secret-scan (push) Successful in 7s
CI / php-lint (push) Successful in 16s
CI / static-tests (push) Successful in 30s
CI / secret-scan (pull_request) Successful in 8s
CI / php-lint (pull_request) Successful in 18s
9c0ea4a345
Corentin merged commit c8f5370cfd into dev 2026-06-15 17:01:11 +02:00
Corentin deleted branch feat/p2-core-restructure 2026-06-15 17:01:11 +02:00
Sign in to join this conversation.
No reviewers
No labels
auto-merge
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: AcadeNice/corentin_wakdo#10
No description provided.