Skip to content

feat: link changelog entries to their pull request - #14

Merged
alexbrndl merged 3 commits into
mainfrom
feature/dcj-185-changelog-liens-vers-les-pull-requests-et-les-auteurs
Aug 11, 2026
Merged

feat: link changelog entries to their pull request#14
alexbrndl merged 3 commits into
mainfrom
feature/dcj-185-changelog-liens-vers-les-pull-requests-et-les-auteurs

Conversation

@alexbrndl

Copy link
Copy Markdown
Owner

DCJ-185. Les entrées de changelog affichaient un identifiant de commit brut, non cliquable.

Avant

- 004c342: Ajoute le canal `postMessage` entre le shell et la preview…

Après

- [#12](…/pull/12) [`004c342`](…/commit/004c342) Thanks [@alexbrndl]()! - Ajoute le canal `postMessage`

Vérifié en générant réellement un changelog, pas en lisant la documentation.

Deux choses découvertes en chemin, documentées

Le générateur exige un jeton GitHub, y compris en local : il interroge l'API pour retrouver la pull request associée à un commit. En intégration continue le workflow le fournit ; à la main il faut GITHUB_TOKEN=$(gh auth token).

Les contrôles de la PR de version doivent être approuvés manuellement. GitHub n'exécute pas les workflows déclenchés par son propre jeton, donc ils restent en action_required et ci-passed n'est jamais rapporté. Constaté sur la PR #13. La parade classique, un jeton personnel, est écartée pour ne pas stocker un secret longue durée sur un dépôt public.

Le remerciement reste, volontairement

Thanks [@alexbrndl] n'a pas de sens sur un projet à un contributeur. Le retirer demande l'option template, qui n'existe qu'à partir de la version 1.0.0 et reste marquée expérimentale. Cette version est assez récente pour être refusée par la politique de fraîcheur des dépendances, et l'installer exigeait trois exclusions.

Prendre une dépendance fraîche et une API instable pour une mention cosmétique est le profil qui a déjà coûté cher avec TypeScript 7. Suivi dans DCJ-187, avec le rendu obtenu et la configuration à remettre le jour venu.

Pas de changeset

Le diff ne touche aucun packages/*/src/** ni champ publié : configuration d'outillage et documentation.

- @changesets/changelog-github, pinned to the catalog at 0.7.0
- document the GitHub token it needs, even locally
- document why the version PR needs manual workflow approval

Refs DCJ-185

@alexbrndl alexbrndl left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

Revue

Verdict : 1 point.

Comment thread docs/architecture.md
@alexbrndl
alexbrndl marked this pull request as ready for review August 11, 2026 16:20
@alexbrndl
alexbrndl merged commit 78503ea into main Aug 11, 2026
6 checks passed
@alexbrndl
alexbrndl deleted the feature/dcj-185-changelog-liens-vers-les-pull-requests-et-les-auteurs branch August 11, 2026 16:28
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