Retour à l'accueil du SAV de la Tech
16. Mon CTO écrit du code pourri 🤢
Cette semaine, dans le SAV de la Tech, on répond à la question de Steeve:
"Petite question de responsabilité, de hiérarchie et de tact.
Je tombe sur un fichier où je trouve des modifications récentes complètement foutraques, plusieurs v-if/v-else/v-show appliqués de manière croisée sur les mêmes éléments de template, créant un bazar illisible et potentiellement bogué, bref, une véritable satanerie qui fait pleurer du sang. Rapidement je me rends compte grâce à gitLens que les modifs ont été faites il y a une semaine par mon CTO 😶. Qu'est-ce que je fais ?
Je fais l'andouille et signale à l'équipe que le code n'est pas correct sur cette feature comme si je n'avais pas vu qui en était l'auteur au risque de l'afficher ? Je fais l'avenger et prends la responsabilité de corriger le code en douce pour épargner son égo au risque de perdre du temps sur ma propre tâche ou de causer moi-même un bug en voulant corriger ? Je fais l'adulte et vais le voir pour lui dire que la feature n'est pas bien codée et qu'il faut la modifier au risque de le vexer et de remettre en question ses compétences ? Je fais l'autruche et passe mon chemin au risque de laisser des bugs partir en prod ?"
Épisode enregistré en Mars 2024.
Crédits musique: "Guess Again", provided by https://slip.stream
Proposer une question, un sujet
Ressources mentionnées dans l'épisode
(🫣 À lire après écoute de l'épisode !)
- Egoless Crafting: Dix proposition pour assainir la collaboration au sein d'équipes de développement. Exemples: éviter le "cargo cult" et questionner les conséquences concrètes (outcomes) plutôt que les opinions.
- Communication Non Violente (CNV): Méthode permettant de résoudre et/ou prévenir les conflits interpersonnels, en traitant séparément les ressentis, faits et besoins, puis en formulant une demande.
- Comment m̶a̶l̶ communiquer avec ses collègues: Talk court qu'Adrien a donné en Mars 2024 sur les erreurs de communication et leadership qu'il a commises lors d'une expérience au sein d'une équipe qu'il a eu du mal à piloter.
- Formation "assertivité pour cadre et manager" - Cegos : Excellente formation suivie par Adrien il y a quelques années. Les techniques de l'assertivité permettent de faire face aux situations de tensions quotidiennes courantes et d'éviter d'en générer soi-même. Cette méthode permet de limiter l'agressivité, la passivité ou la manipulation dans les relations habituelles.
- Unlearning toxic behaviors in a code review culture | by Sandya Sankarram
- How to Make Your Code Reviewer Fall in Love with You · mtlynch.io: Conseils sur comment bien communiquer lors de la revue de code (pull requests ou autres).
- Delegation Poker & Delegation Board - Management 3.0 Practice: Use Delegation Poker to clarify who’s responsible for what and to what level. This is a method where you can encourage employee engagement through controlled self-organization and clarified value and decision-making.