Examiner et traiter les retours de revue
Objectif : Répondre à une remarque de revue par une modification ciblée et vérifier que la discussion porte sur le bon changement.
Une demande de fusion peut recevoir une remarque précise après sa publication : un texte manque de clarté, une information importante n’est pas présente ou un détail doit être corrigé. Si vous modifiez plusieurs éléments sans rapport, la personne qui relit doit reprendre l’ensemble du changement pour vérifier sa remarque.
Vous allez traiter un retour sur la carte « La boucle du lac » de Sentiers. Le commentaire demande de préciser que l’itinéraire convient aux personnes qui débutent. Vous modifiez uniquement la description concernée, puis vous publiez cette correction sur la branche de la demande de fusion afin que le fil de discussion puisse être vérifié sur la bonne version.
Partir du commentaire, pas d’une supposition
Un commentaire de revue signale un point à examiner dans le changement proposé. Lisez-le en entier et repérez le fichier, la ligne ou l’élément auquel il se rapporte. Dans cet exemple, le retour porte sur la description de la carte du lac, pas sur la navigation, la mise en page ni les autres guides.
Avant d’éditer, vérifiez que vous travaillez toujours sur feature/nouveaux-guides, la branche publiée pour la demande de fusion. Vous avez déjà appris à contrôler l’état du dépôt dans la leçon « Lire l’état du dépôt et repérer les branches ». Si le retour est ambigu, demandez une précision dans le fil au lieu d’inventer une exigence ou de modifier une partie différente du site.
Faire une correction ciblée et la contrôler
La remarque concerne une information de contenu : vous ajoutez donc la précision dans le paragraphe de la carte, sans toucher aux autres textes ni aux styles. Le fichier ci-dessous est l’état complet de index.html après la correction. La description devient : « Une promenade paisible au bord de l’eau, accessible aux débutants. »
<!doctype html>
<html lang="fr">
<head>
<meta charset="utf-8">
<meta name="viewport" content="width=device-width, initial-scale=1">
<title>Sentiers — Guides de randonnée</title>
<link rel="stylesheet" href="style.css">
</head>
<body class="atelier-page">
<header class="atelier-entete">
<a class="atelier-marque" href="#accueil">Sentiers</a>
<nav class="atelier-navigation" aria-label="Navigation principale">
<a class="atelier-lien" href="#accueil">Accueil</a>
<a class="atelier-lien" href="#guides">Guides</a>
<a class="atelier-lien" href="#apropos">À propos</a>
</nav>
</header>
<main class="atelier-principal" id="accueil">
<section class="atelier-introduction" aria-labelledby="titre-introduction">
<p>Explorez près de chez vous</p>
<h1 id="titre-introduction">Des idées pour votre prochaine randonnée</h1>
<p>Parcourez des itinéraires choisis pour vous aider à préparer votre prochaine sortie.</p>
<a class="atelier-bouton" href="#guides">Découvrir les guides</a>
</section>
<section class="atelier-fonctionnalite" id="guides" aria-labelledby="titre-guides">
<h2 id="titre-guides">Nouveaux guides</h2>
<div class="atelier-grille">
<article class="atelier-carte">
<h3>Le sentier des crêtes</h3>
<p>Un itinéraire panoramique pour prendre de la hauteur.</p>
<a class="atelier-lien" href="#guide-cretes">Lire le guide</a>
</article>
<article class="atelier-carte">
<h3>La boucle du lac</h3>
<p>Une promenade paisible au bord de l’eau, accessible aux débutants.</p>
<a class="atelier-lien" href="#guide-lac">Lire le guide</a>
</article>
<article class="atelier-carte">
<h3>La forêt des pins</h3>
<p>Un parcours ombragé à découvrir en famille.</p>
<a class="atelier-lien" href="#guide-foret">Lire le guide</a>
</article>
</div>
</section>
</main>
<footer class="atelier-pied" id="apropos">
<p>Sentiers — Des guides pour partir à pied.</p>
</footer>
</body>
</html>Le dépôt ne contient que index.html et style.css. Pour que l’état présenté soit complet, voici aussi la feuille de style associée. Elle conserve les variables et les classes du contrat du projet ; cette correction de revue ne nécessite aucune modification CSS.
:root {
--atelier-couleur-texte: #24312b;
--atelier-couleur-fond: #f7f6f0;
--atelier-couleur-accent: #276749;
--atelier-espace-petit: 0.75rem;
--atelier-espace-moyen: 1.5rem;
--atelier-espace-grand: 3rem;
--atelier-rayon-carte: 0.75rem;
--atelier-typographie: system-ui, sans-serif;
--atelier-largeur-page: 72rem;
}
.atelier-page {
margin: 0;
color: var(--atelier-couleur-texte);
background: var(--atelier-couleur-fond);
font-family: var(--atelier-typographie);
}
.atelier-entete {
display: flex;
justify-content: space-between;
gap: var(--atelier-espace-moyen);
max-width: var(--atelier-largeur-page);
margin: 0 auto;
padding: var(--atelier-espace-moyen);
}
.atelier-marque {
color: var(--atelier-couleur-accent);
font-weight: bold;
}
.atelier-navigation {
display: flex;
flex-wrap: wrap;
gap: var(--atelier-espace-moyen);
}
.atelier-lien {
color: var(--atelier-couleur-accent);
}
.atelier-principal {
max-width: var(--atelier-largeur-page);
margin: 0 auto;
padding: var(--atelier-espace-moyen);
}
.atelier-introduction {
padding: var(--atelier-espace-grand) 0;
}
.atelier-bouton {
display: inline-block;
padding: var(--atelier-espace-petit) var(--atelier-espace-moyen);
color: var(--atelier-couleur-fond);
background: var(--atelier-couleur-accent);
border-radius: var(--atelier-rayon-carte);
}
.atelier-fonctionnalite {
padding: var(--atelier-espace-grand) 0;
}
.atelier-grille {
display: grid;
gap: var(--atelier-espace-moyen);
}
.atelier-carte {
padding: var(--atelier-espace-moyen);
background: var(--atelier-couleur-fond);
border-radius: var(--atelier-rayon-carte);
}
.atelier-pied {
max-width: var(--atelier-largeur-page);
margin: 0 auto;
padding: var(--atelier-espace-moyen);
}Aperçu appliqué au document HTML de cette leçon : la règle CSS est injectée dans la page pour voir son effet réel.
Contrôlez ensuite le diff avant de créer un commit. Il doit montrer la ligne attendue dans la carte du lac et ne pas contenir de changement sans rapport. Le contrôle permet aussi de vérifier que la correction répond réellement au commentaire, plutôt que de simplement ajouter du texte ailleurs.
git diff -- index.html
git add index.html
git diff --staged
git commit -m "Clarifier le public de la boucle du lac"[feature/nouveaux-guides abc1234] Clarifier le public de la boucle du lac
1 file changed, 1 insertion(+), 1 deletion(-)Publier la mise à jour et répondre dans le fil
Le commit local ne modifie pas à lui seul la demande de fusion en ligne. Publiez-le sur la branche de suivi déjà configurée ; la plateforme mettra alors à jour les changements proposés dans la demande. Dans le commentaire, indiquez brièvement ce qui a changé et où, pour aider la personne qui relit à vérifier sa remarque.
git pushMise à jour de origin/feature/nouveaux-guidesRépondez dans le fil de discussion attaché au commentaire, par exemple : « Précision ajoutée à la description de la carte du lac dans le dernier commit ; pouvez-vous vérifier que cela répond à votre remarque ? » Le fil conserve le lien entre le retour et sa correction. Évitez de créer une nouvelle discussion sans lien avec l’ancienne : la personne qui relit pourrait ne pas voir que vous avez traité son point.
À retenir
- Lisez le commentaire et identifiez précisément le contenu qu’il concerne ; demandez une clarification si nécessaire.
- Limitez la correction au point discuté et contrôlez le diff avant de créer un commit.
- Publiez le commit sur la branche de la demande de fusion pour mettre à jour la revue.
- Répondez dans le fil concerné et laissez la validation à la personne qui a formulé la remarque.
Dans la prochaine leçon, vous apprendrez à lire l’historique de commits avant d’en modifier l’organisation.