Je veux pas dénoncer, mais le dernier qui a touché ce bout de code, c'est moi 😉 J'aurais du le faire…
Je voulais parler de logging.disable(logging.CRITICAL)
qui me semble ne pas être au bon endroit.
Sans rapport : visiblement gitea colore mal les multiline strings.
Est-ce qu'il ne devrait pas y avoir un return NotImplemented pour être cohérent avec les autres comparaisons ?
Ça n'est pas forcément sur git.afpy.org, ça dépend de api_url
:wink
Je me suis laissé tromper par le nommage de la variable directory
qui n'est pas un dossier. Ça pourrait être directory_stats
ou dir_stats
ou mieux ?
En ayant écrit une remarque suivante, je remonte ici. Je suis perturbé par le nommage des variables. self.files
n'est pas les fichiers mais les statistiques sur les fichiers. On le voit en lisant la signature de la méthode, mais plus bas dans le code j'avais oublié. Que penses-tu d'un self.files_stats
?
J'imagine qu'il doit y avoir une raison, mais pourquoi ne pas directement pickler dans le fichier souhaité ?
Que penses-tu de https://docs.python.org/3/library/contextlib.html#contextlib.suppress ?
Pourquoi ne pas laisser comme avant ? (J'imagine qu'il y a une bonne raison.)
Est-ce qu'on ne voudrait pas aussi tester l'ordre des informations (ordre des fichiers dans les dossiers, bonne appartenance des fichiers aux dossiers) ? Auquel cas j'imagine une comparaison de toute la sortie d'un coup.
Je me pose la question : est-ce que ça ne devrait pas passer par logging ? Si oui, il faut une configuration par défaut du logging, vu que argparse nous fournira ensuite de quoi le reconfigurer.
C'est bizarre de configurer le logging ici, mais pas pour les autres niveaux de log, non ?
@mdk Dans l'ensemble, ça me semble bien (sacré travail !). J'ai quelques remarques/questions.