Git : 8 réflexes simples avant de pousser du code
Pull Request, Merge Request, Peer Review... Peu importe le nom : au final, vous montrez votre code à l'équipe. Et personne n'aime reviewer un diff incompréhensible un vendredi à 17 h (ni à un autre moment).
Quelques réflexes simples permettent de pousser un code testé, sécurisé et lisible, qui respecte les conventions du projet et le temps de vos collègues.
1. Vérifiez les fichiers sensibles
Pas de appsettings.*.json avec une clé d'API ou un mot de passe dedans. Même « juste pour tester », ça reste dans l'historique Git. Des projets publics sur GitHub se sont fait pirater à cause de vieux commits. Le .gitignore aide, mais relisez toujours votre diff.
2. Nettoyez les logs de debug
Un Console.WriteLine("toto"), c'est inutile. Un LogInformation("token: {jwt}"), c'est dangereux. Nettoyez avant de pousser : ni bruit, ni fuite.
3. Soignez les noms de variables et de méthodes
Une variable temp2 ou une méthode DoStuff(), ça n'aide personne, même un lundi matin. Choisissez des noms précis et explicites, même pour un petit fix.
4. Faites en sorte que le fichier soit propre
Pas de doubles sauts de ligne, pas d'espaces en trop, pas d'indentations foireuses : c'est pénible à relire et ça pollue le diff pour rien. Utilisez Code Cleanup (Visual Studio) pour que tout soit propre à chaque sauvegarde.
5. Soignez vos commits
update, fix truc, final V3... ça ne guide personne. Un bon commit est clair, utile, et lisible comme un historique.
6. Respectez les conventions du projet
Pas de Utils.cs géant qui regroupe tout et n'importe quoi, surtout si le projet a des guidelines claires sur l'architecture. Le formatage, les noms, l'archi : ce n'est pas pour faire joli, c'est pour que dans 3 mois, personne ne rage en lisant votre diff.
7. Testez avant de pousser
Pas « ça marchait chez moi ». Vraiment. Lancez les tests, vérifiez le scénario principal, refaites un tour sur les impacts.
8. Posez-vous LA question
« Est-ce que je trouverais ce code bizarre dans une PR d'un collègue ? » Si la réponse est oui : réécrivez, isolez, ou expliquez.
À retenir : prenez le temps de relire avant de pousser. La qualité d'une PR, c'est aussi ce qui fait la qualité d'une équipe.
Cet article est tiré d'une de mes publications LinkedIn : rejoignez la discussion ↗