Skip to content
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

build(dependabot): Amélioration de la config Dependabot #235

Merged
merged 2 commits into from
Feb 11, 2025

Conversation

jbuget
Copy link
Contributor

@jbuget jbuget commented Feb 10, 2025

Problème

Tous les lundis, il y a plein de nouvelles PR de type "version upatde". Le plus souvent, il s'agit de versions mineures, qui contiennent des micro-fonctionnalités qui ne nous concernent pas ou ne nous sont pas utiles.

Le plus souvent encore, nous faisons l'effort de regrouper ces updates au sein d'une seule et même PR front et une autre back.

Autre souci, par défaut Dependabot limite le nombre de PR ouvertes en même temps à 5 (pour le type version update).

Solution

Dependabot permet de regrouper les version updates au sein de PR uniques (doc).

Cette PR propose d'activer cette fonctionnalité et de regrouper toutes les versions update de type minor ou patch par PR.

Cette PR ne couvre pas les versions majeures, qui continueront d'être proposées régulièrement.

Enfin, cette PR en profite pour régler le problème de nommage du titre des PR qui cassent le linting GitHub Action.

@jbuget jbuget force-pushed the build/dependabot-security-updates branch from 7fc169c to 0c71692 Compare February 10, 2025 17:58
Copy link
Contributor

@ggounot ggounot left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

@jbuget jbuget merged commit 469cb2e into main Feb 11, 2025
6 checks passed
@jbuget jbuget deleted the build/dependabot-security-updates branch February 11, 2025 08:08
@jbuget
Copy link
Contributor Author

jbuget commented Feb 11, 2025

Ca a fonctionné ! cf. la PR #239

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.

3 participants