Refractor: Mise en conformité avec PEP8#34
Merged
AnthonyGuillauma merged 26 commits intodevelopfrom Apr 4, 2025
Merged
Conversation
Owner
AnthonyGuillauma
commented
Apr 4, 2025
AnthonyGuillauma
commented
- Mise en conformité du code avec PEP8
- Déplacement de la classe EntreeLogApache vers le fichier entree_log_apache.py
- Mise à jour de la documentation suite à ces modifications
- Mise à jour des tests unitaires suite à ces modifications
- Ajout d'une action pour tester si la qualité du code est noté correctement par PyLint (>= 9)
- Déplacement de l'appel à la méthode qui lance l'animation de chargement après le parsage des arguments en CLI pour éviter des interférences entre les prints de argparse et cette animation
- Modification de la fonction gestion_erreur pour que les messages ne soient plus décidés par cette fonction - Suppressions de "except Exception" qui est trop générale
- Amélioration des docstrings - Suppression d'espaces inutiles - Ajout de Optional de typing pour typer les paramètres
- Suppressions d'espaces inutiles - Suppression de parenthèses sur certaines conditions if - Rangement des importations
- Déplacement de la classe EntreeLogApache dans un fichier séparé - Mise à jour de la documentation et des tests unitaires suites à ce changement
- Ajout de la classe FichierLogApache comme une dataclass - Ajout de la classe EntreeLogApache comme une dataclass
- Modification des docstrings de FichierLogApache
- Amélioration de la lisibilité de certaines expressions - Suppressions des espaces inutiles - Ajout des méthodes pour extraires les informations afin d'alléger la méthode parse_entree
- Modification des vérifications des types des attributs
- Modification des vérifications des types des attributs
- Modification des vérifications des types des attributs
- Suppression d'espaces blancs inutiles - Suppression des lignes trop longues
- Amélioration des exceptions - Suppression des espaces blancs inutiles - Ajout d'une méthode pour vérifier que l'exportation est possible
- Ajout d'une exception lorsque le fichier de log est introuvable - Définition de la classe ParsageLogApacheException comme la classe mère de toutes les exceptions liées au parsage
- Interception de l'exception ParsageLogApache au lieu des autres sous-classes d'exception liée au parsage pour plus de lisibilité
- Ajout de la précision du type de retour pour toutes les fonctions et méthodes
- Amélioration de la documentation de la constante PATTERN_ENTREE_LOG_APACHE
- Modification de la documentation des attributs des dataclasses pour qu'ils soient bien documentés - Exclusion des doublons de documentation de Sphinx pour les attributs de classe dans les .rst respectifs des dataclasses
- La méthode vérifier que le type du paramètre entree soit de type EntreeLogApache alors qu'il devait de type str
- Mise à jour des tests unitaires suite à la modification du code pour être conforme avec PEP8
- Ajout d'un workflow qui vérifie que la qualité du code est de minimum 9/10 d'après Pylint - Modification du nom du workflow de la documentation
- Ecriture de la note pour débogger ce workflow
- Ecriture du resultat de pylint dans le workflow
- AJout de la dépendance colorama qui cause des erreurs si non présente
- Ajout d'un artefact qui contient les résultats de Pylint dans le workflow qualite
- Changement du nom des artefacts dans les workflows des tests unitaires de la qualite du code
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.