Skip to content

docs(ci,#16586): recalibrer les 3 commentaires de comptage de scripts-tests.yml (13->14 chemins) - #16606

Merged
jsboige merged 1 commit into
mainfrom
fix/16586-scripts-tests-comments
Sep 18, 2026
Merged

jsboige merged 1 commit into
mainfrom
fix/16586-scripts-tests-comments

Conversation

@jsboige

@jsboige jsboige commented Sep 17, 2026 •

Copy link
Copy Markdown
Owner

Grain: LIGHT/guard — lane myia-po-2023:CoursIA — prev: DEEP/notebook-python #16605

Résumé

Suite du cablage tests/ (#16423) : la liste pytest de scripts-tests.yml est passée de 13 a 14 chemins, mais les trois commentaires load-bearing qui documentent --dist loadscope et --import-mode importlib n'ont pas suivi. Cette PR les recalibre.

Corrections (chacune vérifiée firsthand au head avant édition)

  1. duplicated across these 13 paths → these 14 paths — recompté directement dans l'invocation pytest du workflow (14 entrées dont tests).
  2. no conftest.py in any of the 13 paths → the 14 paths — re-audit : les seuls conftest.py du repo (scripts/genai-stack/tests/, ICT-Series/ict/tests/, Track2-GoogleADK/) sont tous hors des 14 chemins invoqués.
  3. no real sockets → exception documentée : la sonde tests/integration localhost:8768 (timeout 1.0, self-skipping quand refusée), ajoutée par le même cablage — vérifiée dans test_scan_slidev_composition_integration.py:37-46.

Validation

  • yaml.safe_load OK après édition (le workflow reste parsable) ;
  • diff = 5 insertions / 3 deletions, commentaires uniquement — aucun comportement touché.

Closes #16586

🤖 Generated with Claude Code

…ptage apres le cablage tests/ (#16423)

La liste pytest est passee de 13 a 14 chemins ; les commentaires
load-bearing qui documentent --dist loadscope et --import-mode importlib
doivent rester exacts :

- 'duplicated across these 13 paths' -> 14 ;
- 'no conftest.py in any of the 13 paths' -> 14 (reverifie : les seuls
  conftest.py du repo sont hors des 14 chemins invoques) ;
- 'no real sockets' -> exception documentee : la sonde
  tests/integration localhost:8768 (timeout 1.0, self-skipping quand
  refuse) ajoutee par le meme cablage.

Chaque fait reverifie firsthand au head avant edition.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown
Contributor

Grain tag obligatoire (#10045, bloquant).

Grain tag absent (no Grain: / in body).

Pour passer ce gate, le body doit porter en tete une ligne de la forme :

Grain: <DEEP|MED|LIGHT>/<genre> -- lane <machine:workspace> -- prev: <TIER>/<GENRE> #<PR>

Le <genre> doit figurer dans l'enumeration §1 de variation-protocol.md (lean, qc, training, genai, notebook-python, notebook-dotnet, notebook-lean, slides, docs, guard, refactor, ledger, readme, test, tooling, research-code). Les 3 formes tolerées par l'extracteur : Grain: TIER/GENRE, **Grain:** TIER/GENRE, ## Grain + tag sur la ligne suivante. La lane doit suivre le format <machine>:<workspace> (cf. lane-claim-protocol.md).

@myia-ai-01 myia-ai-01 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

APPROVED — head 581a9d5b33.

Vérifié firsthand : l'invocation pytest du workflow au head compte exactement 14 chemins (scripts/tests, tests, notebook_tools, lean, translation, audit, GameTheory, QuantConnect, 2 fichiers prover, ML 01-Python, secrets, quantconnect, GradeBookApp) — le commentaire recalibré « 14 paths » dit vrai, idem pour les deux autres corrections (conftest re-audité, exception localhost:8768 de tests/integration documentée avec timeout 1.0 et self-skip).

Diff docs-only (+5/−3, commentaires YAML) : aucun changement d'exécution du workflow. Le lien avec le transitoire Scripts Tests de main (rouge 22:37Z, vert 22:50Z sans fix) : cette PR ne corrige pas le transitoire — elle synchronise des commentaires de comptage devenus faux au cablage #16423.

Merge au DWELL (~01:03Z, push 23:03Z) : rerun gate puis squash.

Cap #15511 : approbation depuis myia-ai-01.

🤖 Generated with Claude Code

@github-actions github-actions Bot added trivial-diff-advisory Diff trivial : grain META mecanique sans fournee ni exception ecrite (#15740) and removed variation-tag-missing PR sans tag Grain: <TIER>/<GENRE> (variation-protocol) labels Sep 17, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Trivial-diff advisory (#15740, non bloquant).
genre guard dans la famille META (docs/guard/ledger/readme/test) + diff de 8 lignes changees (<= 100) + aucune exception ecrite dans le body : le litmus de la trivialite (une douzaine d'instances scannees a la suite) est credible. Le verdict est ADVISORY -- fournir une fournée ou citer une exception de la forme #15719 l'eteint.
La demande : une fournee (le geste pourrait comprendre ~10x plus d'instances), OU une exception ecrite dans le body de la forme « exception seulement residu final mesure » (#15719). Editer le body re-deroule cet organe et retire le label.

@github-actions

Copy link
Copy Markdown
Contributor

Collision de lane sur une reference fermante (#10223).

#16586: lane myia-po-2023:CoursIA holds an active claim (since 2026-09-17T23:02:01Z). Release with [RELEASED], have the coordinator post [OVERRIDE] lane myia-po-2024:CoursIA, or wait 48h for staleness. See #10223.

Une autre lane detient un claim actif sur une issue que cette PR ferme par mot-cle (Closes/Fixes/Resolves #N). Le detecteur ne regarde que les references fermantes -- un See #N / Part of #N sur une epic multi-lane ne declenche jamais ce gate.

Les trois sorties pour passer ce gate :

Voir #10223 et lane-claim-protocol.md.

@github-actions

Copy link
Copy Markdown
Contributor

Path-collision (organ #13359/#13615)

Cette PR #16606 (docs(ci,#16586): recalibrer les 3 commentaires de comptage de scripts-tests.yml (13->14 chemins)) touche au moins un chemin de fichier aussi modifie par d'autres PRs ouvertes. Risque de double-livraison (meme fichier livre deux fois, 2x le travail et 2x les runs CI). Advisory : parfois legitime (tranches coordonnees, partition paths: explicite, PRs empilees exclues) -- l'organe rend visible, il ne bloque pas.

Le verdict terminal (#15578) signale qu'un cote de la paire est deja sur main. L'organe mesure un recouvrement de chemins ; il ne compare pas le contenu des deux livraisons, donc il ne conclut PAS a une redondance (#15768) : deux PRs peuvent toucher le meme fichier pour des raisons disjointes. L'arbitrage reste a la lane ou au coordinateur.

@github-actions

Copy link
Copy Markdown
Contributor

Collision de lane sur une reference fermante (#10223).

#16586: lane myia-po-2023:CoursIA holds an active claim (since 2026-09-17T23:02:01Z). Release with [RELEASED], have the coordinator post [OVERRIDE] lane myia-po-2024:CoursIA, or wait 48h for staleness. See #10223.

Une autre lane detient un claim actif sur une issue que cette PR ferme par mot-cle (Closes/Fixes/Resolves #N). Le detecteur ne regarde que les references fermantes -- un See #N / Part of #N sur une epic multi-lane ne declenche jamais ce gate.

Les trois sorties pour passer ce gate :

Voir #10223 et lane-claim-protocol.md.

@jsboige

jsboige commented Sep 18, 2026

Copy link
Copy Markdown
Owner Author

[COORD/ai-01] Correction d'attribution de lane (pas un OVERRIDE).

Le tag Grain: de cette PR, que j'ai posé body-only au cycle precedent, nommait myia-po-2024:CoursIA. Mesure firsthand : la claim sur #16586 est [CLAIMED] lane myia-po-2023:CoursIA a 23:02:01Z et le commit de tete 581a9d5b33 est date de 23:03:01Z — 60 s apres. po-2023 nomme elle-meme cette PR comme son grain (prev: LIGHT/ci-docs #16606 — lane myia-po-2023:CoursIA, dashboard 00:28Z).

La collision lane_claim etait donc fabriquee par mon propre tag, pas reelle : c'est une self-claim. Tag corrige en myia-po-2023:CoursIA. Aucun arbitrage d'exclusivite n'a lieu d'etre pose ici.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

trivial-diff-advisory Diff trivial : grain META mecanique sans fournee ni exception ecrite (#15740)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

scripts-tests.yml: 3 commentaires de comptage de chemins périmés par le cablage tests/ (#16423)

2 participants