Skip to content

📝 docs(release): corrige quatre passages démentis par le cut de la v1.0.1 - #45

Merged
kbrdn1 merged 4 commits into
devfrom
docs/#43-release-md
Sep 19, 2026
Merged

kbrdn1 merged 4 commits into
devfrom
docs/#43-release-md

Conversation

@kbrdn1

@kbrdn1 kbrdn1 commented Sep 19, 2026

Copy link
Copy Markdown
Owner

Summary

La release v1.0.1 a exécuté chaque étape de docs/RELEASE.md pour de vrai. Quatre passages ne correspondaient pas à ce qui s'est passé : cette PR les corrige, sans rien changer d'autre.

Related Issue

Closes #43

Type of Change

  • 🐛 Bug fix (non-breaking change which fixes an issue)
  • ✨ New feature (non-breaking change which adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • 📝 Documentation update
  • ♻️ Refactoring (no functional changes)
  • ⚡️ Performance improvement
  • 🎨 Style/UI update

Scope

Which parts are affected?

  • Homepage
  • Blog
  • Projects
  • UI Components
  • i18n / Translations
  • SEO
  • Configuration

Checklist

  • My code follows the project's coding style
  • I have performed a self-review of my code
  • I have tested my changes locally
  • My changes generate no new warnings
  • I have updated the documentation if needed
  • TypeScript compiles without errors (make typecheck)
  • Linting passes (make lint)
  • Formatting passes (bun run fmt:check)

Cases de tests, typecheck, lint et format non cochées : la PR ne touche qu'un .md, que le format exclut. Rien à lancer localement.

Screenshots (if applicable)

N/A

Additional Notes

Chaque correction s'appuie sur une source vérifiable :

Passage Avant Source
Linked issue « toute PR vers main » validate-pr.yml : branches: [main, dev], et linked-issue n'a pas de condition sur la base ; rouge sur #40 avec Refs #39
Étape 4 « attendre les 4 checks, puis merger » protection de main : required_approving_review_count: 1, enforce_admins: false ; #29 / #32 / #41 mergées avec REVIEW_REQUIRED
Étape 6 /api/health comme vérification nuxt.config.ts : appVersion: APP_VERSION || pkg.version, et deploy.yml ne passe pas d'APP_VERSION ; health à 1.0.1 pendant l'échec du run 35420011269
Revenir en arrière « reconstruit l'image et tente de recréer une release » release.yml : step Image already published? (build sauté), puis gh release view → gh release edit

Les commandes ajoutées à l'étape 6 ont été lancées sur la v1.0.1 : run success, et github-actions[bot] false.

https://claude.ai/code/session_01P5DUscqftnE27pbmuUqL5J

….0.1

- Linked issue s'applique aux PR vers dev aussi (validate-pr.yml n'a
  aucune condition sur la base) ; Refs ne passe pas ; éditer le corps
  relance le check, un rerun rejoue l'ancien corps.
- Étape 4 : main exige une approbation en plus des checks ; les
  promotions passent en --admin (enforce_admins à false).
- Étape 6 : /api/health ne distingue pas l'image du tag de celle du
  filet deploy.yml ; la release se vérifie sur son run et gh release view.
- « Revenir en arrière » affirmait qu'un tag rejoué reconstruit l'image
  et recrée la release : release.yml saute le build et fait un edit.
  L'étape 5 disait déjà l'inverse, le fichier se contredisait.

Plus le rattrapage d'un Deploy prod en échec (#42) et une référence
morte (« cf. contrainte 2 »).

closes #43

Claude-Session: https://claude.ai/code/session_01P5DUscqftnE27pbmuUqL5J
@kbrdn1 kbrdn1 added the docs Documentation label Sep 19, 2026
…ejoue les notes du tag

Le bloc sur l'approbation coupait la phrase sur required_linear_history,
dont le « pour ça » renvoyait alors au merge admin ; il passe après, la
phrase d'origine redevient intacte. `--admin` ignore aussi les checks
requis : seule l'attente des checks verts protège. Un tag rejoué remet
les notes de l'arbre du tag, pas une correction faite depuis. « Toutes
les promotions » devient la liste vérifiée (#29, #32, #41).

closes #43

Claude-Session: https://claude.ai/code/session_01P5DUscqftnE27pbmuUqL5J
…run du tag

L'étape 5 disait « sans effet de bord » là où « Revenir en arrière » dit
que les notes sont réécrites depuis le tag : l'étape 5 renvoie désormais
à ce paragraphe. `gh run list --branch vX.Y.Z` isole le run du tag (un
dispatch s'affiche sous main). L'ordre filet / tag dépend du verrou, et un
cut avec candidat a plus de deux PR : les deux formulations sont
corrigées. L'avertissement --admin tient en une phrase.

closes #43

Claude-Session: https://claude.ai/code/session_01P5DUscqftnE27pbmuUqL5J
…n du tag

`gh run list --branch vX.Y.Z` ne voit pas un rattrapage en
workflow_dispatch (rangé sous main) : sur la v1.0.0 il ne montre que le
run en échec alors que la release est publiée. Le job release dépend de
deploy, qui fait le contrôle /api/health : `gh release view` suffit. Le
titre de la contrainte dit ce que le check exige (un mot-clé de
fermeture), pas une fermeture.

closes #43

Claude-Session: https://claude.ai/code/session_01P5DUscqftnE27pbmuUqL5J
@kbrdn1
kbrdn1 merged commit 8040551 into dev Sep 19, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant