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

Update rule of commit #603

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Update rule of commit #603

wants to merge 3 commits into from

Conversation

pplancq
Copy link
Contributor

@pplancq pplancq commented Nov 19, 2024

Règles de Commit

Convention de Commit

Nous utilisons Conventional Commits pour nous assurer que les messages de commit respectent les conventions spécifiées.

<type>[scope]: <description>

[optional body]

[optional footer(s)]

Voici les règles à suivre pour les scopes de commit :

  • Utilisez le scope correspondant au design system mis à jour par le commit. Les scopes disponibles sont :
    • slash
    • look&feel
    • apollo
  • Si le commit traite de deux ou plusieurs design systems, séparez les scopes par une virgule (,).
  • Si le commit est plus général au projet, utilisez le scope design-system.
  • Les scopes deps, deps-dev et release sont réservés et ne doivent pas être utilisés.

Exemples de messages de commit valides :

feat(slash): ajout d'un nouveau composant
fix(look&feel,apollo): correction des styles et des icônes
chore(design-system): mise à jour des dépendances

Breaking Changes

Pour indiquer un changement majeur qui casse la compatibilité, ajoutez un '!' entre le scope et les ':' ainsi qu'une section "BREAKING CHANGE" dans le footer du commit.

Le '!' permet de signaler que le commit contient une breaking change.
La section "BREAKING CHANGE" permet de décrire les changements majeurs apportés par le commit.

Exemple :

feat(slash)!: suppression des classModifiers

BREAKING CHANGE: Les classModifiers ont été supprimés des composant Slash. Il faut désormais utiliser les props dédiés comme `variant` ou `size`.

@pplancq
Copy link
Contributor Author

pplancq commented Nov 19, 2024

On peut voir que la CI fait bien son office et analyse bien l'ensemble des commit de la PR

image

Fake.md Outdated Show resolved Hide resolved
@pplancq pplancq self-assigned this Dec 10, 2024
@pplancq pplancq added the chore Issue link to publishing, etc label Dec 10, 2024
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
60.0% Coverage on New Code (required ≥ 80%)
7.5% Duplication on New Code (required ≤ 3%)
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Issue link to publishing, etc
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants