Ce donner des règles de programmation et les enforcer par la CI c'est utile aussi.
Pour moi :
les fonctions de 5 kilomètres
C'est une remarque qui ne devrait pas exister en revue. Si tu as passé quelques secondes pour lire/remarquer/faire un commentaire sur quelque chose qui peut tout à fait être automatisé, tu as cassé le fil de ta réflexion pour faire des remarques à plus haute valeur ajoutée. L'idée c'est que si la revue sert à faire linter comme peuvent le faire les linters, les outils d'analyse statique ou les éditeurs évolués tu as perdu du temps et de l'énergie sur quelque chose qui peut apporter plus à tout le monde.
En gros, passer par du code neutre (qui n’était fait par personne dans la boite) a permis de virer l'aspect affectif dans la critique du code et sortir des règles validées par tout le monde.
C'est pas mal établir les règles de programmation. Merci pour l'astuce.
[^] # Re: Tu n'es en rien en voie d'extinction
Posté par barmic 🦦 . En réponse au journal Je fais partie d'une espèce menacée d'extinction. Évalué à 5.
Ce donner des règles de programmation et les enforcer par la CI c'est utile aussi.
Pour moi :
C'est une remarque qui ne devrait pas exister en revue. Si tu as passé quelques secondes pour lire/remarquer/faire un commentaire sur quelque chose qui peut tout à fait être automatisé, tu as cassé le fil de ta réflexion pour faire des remarques à plus haute valeur ajoutée. L'idée c'est que si la revue sert à faire linter comme peuvent le faire les linters, les outils d'analyse statique ou les éditeurs évolués tu as perdu du temps et de l'énergie sur quelque chose qui peut apporter plus à tout le monde.
C'est pas mal établir les règles de programmation. Merci pour l'astuce.
https://linuxfr.org/users/barmic/journaux/y-en-a-marre-de-ce-gros-troll