-
Notifications
You must be signed in to change notification settings - Fork 24
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
Générer le changelog automatiquement depuis GitHub [GEN-1526] #3952
Conversation
0c01344
to
430531d
Compare
430531d
to
1af1d03
Compare
0b3235d
to
f00b9e0
Compare
f00b9e0
to
7eae803
Compare
185efa2
to
edf5547
Compare
This comment was marked as outdated.
This comment was marked as outdated.
594ec01
to
f5b635f
Compare
Je viens de corriger le tri pour ignorer le markdown. |
51bcf1d
to
3515d81
Compare
f43bf19
to
4217863
Compare
) | ||
pull_requests = json.loads(result.stdout) | ||
if len(pull_requests) == limit: | ||
sys.exit("Limit has been exceeded when fetching pull requests, keep up the good work!") |
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.
1000 PR en une semaine, ça laisse un peu de marge ! 😄
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.
Y'a eu débat sur le nombre de zéro, il faut savoir que François est ambitieux 🥲
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.
J'aime ! 😍
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.
Au top 🔥
.github/workflows/require-labels.yml
Outdated
- name: Require minimum one label | ||
uses: mheap/github-action-required-labels@v5.4.0 | ||
with: | ||
mode: minimum | ||
count: 1 | ||
labels: "ajouté, modifié, supprimé, dependencies, no-changelog" | ||
|
||
- name: Require at most one label | ||
uses: mheap/github-action-required-labels@v5.4.0 | ||
with: | ||
mode: maximum | ||
count: 1 | ||
labels: "ajouté, modifié, supprimé, dependencies, no-changelog" |
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.
On doit pouvoir simplifier comme ça je pense
- name: Require minimum one label | |
uses: mheap/github-action-required-labels@v5.4.0 | |
with: | |
mode: minimum | |
count: 1 | |
labels: "ajouté, modifié, supprimé, dependencies, no-changelog" | |
- name: Require at most one label | |
uses: mheap/github-action-required-labels@v5.4.0 | |
with: | |
mode: maximum | |
count: 1 | |
labels: "ajouté, modifié, supprimé, dependencies, no-changelog" | |
- name: Require exactly one label | |
uses: mheap/github-action-required-labels@v5.4.0 | |
with: | |
mode: exactly | |
count: 1 | |
labels: "ajouté, modifié, supprimé, dependencies, no-changelog" |
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.
Bien vu, merci 👍
scripts/mkchangelog
Outdated
sprint_end = sprint_start + datetime.timedelta(days=7) | ||
|
||
if missing := list_missing_pull_requests(sprint_start, sprint_end): | ||
missing_pull_requests = "\n -".join(pr["url"] for pr in missing) |
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.
missing_pull_requests = "\n -".join(pr["url"] for pr in missing) | |
missing_pull_requests = "\n- ".join(pr["url"] for pr in missing) |
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.
L’œil de lynx 😁
4217863
to
694743d
Compare
a41af0c
to
89ae710
Compare
Je viens d’ajouter |
Et évidemment... mheap/github-action-required-labels#66 |
Use the `gh` tool, it’s available by default on GitHub actions.
89ae710
to
a479e93
Compare
🤔 Pourquoi ?
Moins de travail manuel.
🍰 Comment ?
Utilisation de l’outil en ligne de commande
gh
, qui est disponible par défaut sur les machines des GitHub actions.Petite tristesse lorsque je me suis aperçu que l’API pull requests ne permet pas de filtrer par date, le script doit donc itérer sur toutes les pull requests. 😢https://docs.github.com/en/rest/pulls/pulls?apiVersion=2022-11-28#list-pull-requests--parameters
Merci Léo, qui m’a montré l’API
gh search
:)Ajout d’une action pour vérifier la cohérence des labels des pull requests.