RSS Amplifier

Redjista Logbook · Jan 6, 2024

GitHub: Éxiger au moins une approbation avant de pouvoir merge une Pull Request

0
Sign in to vote or save

Julien Leleu · Redjista Logbook

Cela pourrait être votre souhait, en tant que Tech Lead par exemple, d’exiger que chaque Pull Request soit review par au moins une personne (sur les branches Git majeures).

Eh bien, il est possible de configurer un repository GitHub de telle manière à ce que le merge de PR ne soit techniquement pas possible sans l’approbation d’un autre développeur.

Cela prend la forme suivante (message clair en rouge et bouton grisé) :

  1. Rendez-vous dans les “Settings” du repository

  2. Dans la section “Code and automation”, cliquez sur “Branches”

  3. Puis cliquez sur “Add rule”

  4. Dans “Branch name pattern”, saisissez un nom de branche ou une expression régulière qui cible les branches que vous souhaitez protéger

    1. tip : utilisez les wildcards astérisque (*) et point d’interrogation (?)

    2. exemple : release* pour cibler les branches qui commencent par “release”

  5. Cochez d’abord la case “Require a pull request before merging”

  6. Puis cochez la case “Require approvals“

La select box vous permet de choisir le nombre d’approbations requises (ici, 1)
  1. Conseil : cochez également la case “Do not allow bypassing the above settings”, sans quoi il suffira à n’importe qui d’un seul clic pour s’affranchir de la règle

Bon, l’idée reste quand même de sensibiliser l’équipe au cadre et à l’organisation en place. Les outils sont là pour aider mais on peut toujours les by-passer.

Vous voilà quand même un peu plus protégé contre les merge sauvages.

✌️

No posts

Read the original on julienleleu.substack.com

Comments

Nothing yet. Say the first thing.

    Sign in to join the conversation.