David a rencontré une demande de tirage (pull request) déroutante avec des commentaires étendus axés sur un changement apparemment simple : l'ajout d'un seul paramètre à une fonction JavaScript. La fonction d'origine avait déjà sept paramètres, ce qui suggérait potentiellement une opportunité de refactoring pour utiliser un objet. La révision initiale a introduit une instruction conditionnelle liée à un nouveau paramètre, notArg8. Si notArg8 était vrai, le code passait arg8: !notArg8 au contexte. L'auteur ne pouvait pas comprendre cette approche compliquée, car il aurait suffi d'ajouter simplement le paramètre. David a commenté le mauvais code, mais le développeur d'origine a cessé de répondre. Un développeur junior a approuvé le changement de code confus, mais il n'a jamais été fusionné. Un autre développeur a ensuite simplifié la solution, en ajoutant simplement arg8 et en le passant au contexte. Le code simplifié a fonctionné, a passé les tests et a rendu tout le monde heureux. L'intention du développeur d'origine et la raison derrière l'instruction conditionnelle compliquée restent un mystère. La fin suggère que la vision du développeur d'origine reste irréalisée.
notArg8. SinotArg8était vrai, le code passaitarg8: !notArg8au contexte. L'auteur ne pouvait pas comprendre cette approche compliquée, car il aurait suffi d'ajouter simplement le paramètre. David a commenté le mauvais code, mais le développeur d'origine a cessé de répondre. Un développeur junior a approuvé le changement de code confus, mais il n'a jamais été fusionné. Un autre développeur a ensuite simplifié la solution, en ajoutant simplementarg8et en le passant aucontexte. Le code simplifié a fonctionné, a passé les tests et a rendu tout le monde heureux. L'intention du développeur d'origine et la raison derrière l'instruction conditionnelle compliquée restent un mystère. La fin suggère que la vision du développeur d'origine reste irréalisée.