Bof, pour 150 lignes de code. Et y’a un ou deux endroits où j’ai refactorisé. Mais je connaissais pas, donc merci ; en fait j’en ai jamais eu besoin, ça me prend jamais plus de 5 minutes.
Tu as aussi flake8 qui fait non seulement le boulot de pep8, mais qui te permet également de détecter des erreurs un peu bêtes comme une variable déclarée et jamais utilisée, ou bien une variable utilisée sans jamais avoir été affectée, ou des symboles importés et utilisés nulle part…
En fait, c’est tellement pratique que je l’ai directement intégré à mon éditeur de texte : dès que je sauvegarde un fichier Python, il me sort la liste des trucs à corriger.
Quant à la limite de colonnes, un petit indicateur visuel dans l’éditeur ne peut pas faire de mal (ici la colonne en rouge à droite) :
Edit : Avant que quelqu’un ne ramène pylint sur le tapis, je tiens à rappeler que pylint est le seul soft ayant réussi à passer le test de Turing en se faisant passer pour un autiste.
Clair, perso j’essaye toujours d’avoir un linter pour les langages que j’utilise. Le seul défaut c’est que des fois ça consomme bien trop de ressources pour un plugin, mais qu’est ce que c’est pratique. (bon j’avoue que je code en majorité sur Atom pour le moment et que je n’utilise plus trop vim)
Pour l’indicateur visuel, j’en ai déjà trois (80, 100, 120). Quand je code, je me restreins softement à 80 et strictement à 100 (en Python en tout cas) mais pour rester dans les clous, je suis les conventions quand je push/PR.
Edit : Avant que quelqu’un ne ramène pylint sur le tapis, je tiens à rappeler que pylint est le seul soft ayant réussi à passer le test de Turing en se faisant passer pour un autiste.
Bah, il est pas si terrible que ça. Une fois configuré proprement, il sort pas (trop) de faux positifs.
Le problème c’est que (en particulier sur un système de CI) les faux positifs sont une plaie encore plus insupportable que les faux négatifs.
J’ai longuement utilisé pylint, n’ayant pas peur des environnement spartiates, mais j’ai arrêté du jour où je me suis rendu compte que je passais plus de temps à configurer pylint ou mettre des commentaires dans le code pour désactiver telle ou telle erreur en justifiant à chaque fois la désactivation.
Je trouve ça contre-productif de devoir perdre du temps à se justifier en commentaires au lieu de faire du code. Et puis ça crible littéralement le code de commentaires dégueus.
Ouais, c’est un point de vue qui se défend (encore plus dans un cadre pro, j’imagine). Perso je m’en fous d’avoir des faux positifs (tant qu’il n’y en a pas un par ligne, bien sûr :p).
Le boulot ils vont se faire voir aujourd’hui, j’ai toujours pas digéré la nouvelle de ce matin disant que tout le boulot que j’ai fais depuis un mois et demi sert à rien.
À vrai dire, c’est surtout pour savoir qui placer dans la liste « Si tu avais le pouvoir de bannir un membre définitivement, lequel choisirais-tu ? » des zawards 2017. D’ailleurs, la modification du style pour la Saint-Valentin, ça va dans « L’innovation négative de ZdS en 2017 » ou dans « L’incident technique qui aura fait le plus ch*er en 2017 » ?
Connectez-vous pour pouvoir poster un message.
Connexion
Pas encore membre ?
Créez un compte en une minute pour profiter pleinement de toutes les fonctionnalités de Zeste de Savoir. Ici, tout est gratuit et sans publicité.
Créer un compte