-
Notifications
You must be signed in to change notification settings - Fork 10
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ci: Github Action pour lancer release-please #4701
Conversation
release-please: | ||
runs-on: ubuntu-latest | ||
steps: | ||
- uses: GoogleCloudPlatform/[email protected] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
il y a une v4, mais ca change en profondeur la façon de configurer le changelog (changelog-types
), j'upgraderais quand je saurais faire (en le mettant en place sur un autre projet avant !)
.github/workflows/release-please.yml
Outdated
changelog-types: | | ||
[ | ||
{"type":"feat","section":"Nouveautés","hidden":false}, | ||
{"type":"refactor","section":"Améliorations","hidden":false}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Je le trouve pas intuitif que refactor = Améliorations. Dans ma tête refactor c'est un changment du code sans chngement pour les utilisateurs et améliorations c'est des améliorations du produit qui impactent l'utilisateur. Est-ce qu'on peut voir les deux options ? Genre refactor
est mappé vers Technique
et on a un autre type
pour les améliorations produit ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
en effet y'a 2 possibilités :
- soit on rebascule
refactor
dans Technique, et on regroupe les Nouveautés et Améliorations viafeat
- soit on utilise d'autres tokens comme
perf
pour du refactoring pure code
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Dans tous les cas, je pense qu'on aura toujours une petite étape pour retravailler la release une fois publiée dans github, pour ensuite la copier-coller et la partager au reste de l'équipe. Et à ce moment là séparer les Nouveautés des Améliorations, ou au moins regrouper par fonctionnalité
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
il y a une discussion ici pour rajouter improve
par exemple : conventional-commits/conventionalcommits.org#66
je vais voir si on peut le rajouter as-is
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
en tout cas la Github Action https://github.com/amannn/action-semantic-pull-request a l'air de pouvoir customiser la liste des des "types"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
du coup je vais :
- re-basculer "refactor" dans la section Technique (done ici : 20e05ad)
- merger la PR
- voir ce que ca donne, en discuter avec l'équipe, et refaire une PR supplémentaire pour rajouter "improve"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(dernière) PR ouverte ici : #4758
1269d2b
to
ab0b261
Compare
Quoi
Dernière brique du switch vers release-please (cf #4379, suite de #4700) :
à chaque fois qu'une PR sera mergée, cette CI se lancera et mettra à jour une PR de release
Configuration
En regardant un peu nos releases historiques, j'ai séparé les changements en 4 sections :
Plus d'infos
Voir l'issue #4379 :)