Proposer un changement (PR — Pull Request)

En une phrase : créer une branche, faire le changement, et demander à l'administrateur de relire avant fusion (merge) avec le cahier principal (main). Difficulté : attention — flux multi-étapes Temps : 10 à 30 minutes selon la taille du changement

Quand l'utiliser

• Un changement non trivial (> 10 lignes modifiées, ou logique métier sensible)
• Un changement risqué (auth, deploy, Firebase rules, build CI…)
• Vous voulez l'avis de l'administrateur avant de fusionner

Pour les corrections de typo, petits fix locaux, vous pouvez commit + push direct sur main (cf. sauvegarder-mes-etapes.md).

Avant de commencer

• [ ] Vous êtes à jour avec main (git pull récent)
• [ ] Le sujet est défini (1 changement = 1 PR)
• [ ] Le code que vous modifiez compile

Étapes

1. Créer un cahier de brouillon (branche)

git checkout -b fix/affichage-quantite-zero

Convention de nommage : <type>/<sujet-court>. Types : feat, fix, refactor, test, docs, chore.

2. Faire vos changements

Coder, tester localement. Builder. Lancer les tests (/test).

3. Sauvegarder et envoyer le brouillon au cloud

git add -A
git commit -m "fix(dpgf): afficher 0 au lieu de vide sur lots non quantifiés"
git push -u origin fix/affichage-quantite-zero

-u origin <branche> lie votre branche locale à la même branche distante. Aux push suivants, juste git push.

4. Créer la demande de relecture (PR)

gh pr create --title "fix(dpgf): afficher 0 au lieu de vide sur lots non quantifiés" --body "@'
## Résumé
- 1 fix sur DpgfNativeWindow.xaml.cs ligne 487
- Avant : cellule vide. Après : '0'.

## Plan de test
- [ ] Ouvrir 72_HLB_PRO_B, sélectionner lot vide, vérifier que la colonne Quantité affiche 0
- [ ] Vérifier que les lots remplis ne sont pas affectés

🤖 Generated with Claude Code
'@"

Sortie : une URL https://github.com/91STUDIO971/auguste-revit/pull/N → ouvrez-la.

5. Attendre la CI (robot vérificateur)

Sur la page de la PR, GitHub Actions lance les tests :

• vert → vous pouvez demander à l'administrateur de relire
• rouge → corrigez d'abord (push direct sur la branche, la PR se met à jour automatiquement)

6. demander à l'administrateur de relire

Mention @admin dans un commentaire de la PR, ou SMS si urgent.

7. Fusion (merge) dans le cahier principal

Une fois la PR approuvée :

# Méthode 1 — depuis l'interface GitHub
# Cliquer "Squash and merge" (preferred) ou "Merge"

Ou en ligne de commande :

gh pr merge --squash --delete-branch

--squash regroupe tous vos commits en un seul → historique propre sur main. --delete-branch supprime la branche distante (et propose de supprimer la locale).

8. Retour sur main

git checkout main
git pull
git branch -d fix/affichage-quantite-zero   # supprime la branche locale

Schéma complet

Vérifier que ça a marché

• Sur GitHub, la PR a un badge vert "Merged"
• Votre changement apparaît dans git log de main (après git checkout main && git pull)
• L'auto-update va le propager aux postes utilisateurs au prochain démarrage de Revit

Si ça ne marche pas

Symptôme	Que faire
CI rouge persistante	Voir le détail (gh run view --log-failed). Si vous ne comprenez pas, demander à l'administrateur.
Conflit avec main	git checkout main && git pull, puis git checkout fix/X && git rebase main. Résoudre les conflits, push.
gh: command not found	Installer GitHub CLI : winget install GitHub.cli puis gh auth login

Ce qu'il ne faut JAMAIS faire

• Force-push (git push -f) sur main — historique perdu pour tout le monde
• Fusionner une PR avec CI rouge sans investigatio l'administrateur
• Mélanger plusieurs sujets dans une seule PR — la rendre impossible à relire proprement

---

Public cible : développeur futur Dernière vérification : 2026-05-15