Fix pre-commit config and run on all files #103
Conversation
There was a problem hiding this comment.
C'est surtout de la mise en page, et c'est surtout une question de goût.
Je préfère généralement une seule ligne, sauf s'il y a vraiment trop de paramètres dans les parenthèses/crochets, mais je suis étonné que des fois les modifications vont dans ce sens (mettre sur une ligne), des fois dans l'autre sens (éclater la parenthèse sur 3 lgines).
Peut-être utilises-tu "black" ? Je sais qu'il a tendance à faire ce genre de choses
| os.makedirs(self.paths.updated_las_dir, exist_ok=True) | ||
| self.out_las_filepaths = [osp.join(self.paths.updated_las_dir, osp.basename(f)) for f in self.las_filepaths] | ||
| self.out_las_filepaths = [ | ||
| osp.join(self.paths.updated_las_dir, osp.basename(f)) for f in self.las_filepaths |
There was a problem hiding this comment.
Dans les autres fichiers que j'ai vus, beaucoup des modifications étaient de basculer sur une ligne. Ici c'est l'inverser, il y a un retour à passer les "list comprehension" sur 3 lignes.
Personnellement je préfère sur 1 ligne, je trouve que c'est plus lisible. Je suppose que c'est une question de goût, mais c'est surtout que les changements vont un coup dans un sens un coup dans l'autre qui me fait écrire ce commentaire
Standardize format using git pre-commit