Merged
Conversation
Owner
AnthonyGuillauma
commented
Apr 3, 2025
AnthonyGuillauma
commented
- Ajout de la classe AfficheurCLI qui contient les méthodes pour intéragir avec la CLI
- Ajout des tests unitaires d'AfficheurCLI
- Ajout de la documentation d'AfficheurCLI
- Ajout de la classe AfficheurCLI qui permet d'intéragir avec la ligne de commande et d'afficher l'animation de chargement - Ajout du dossier assets qui contient tous les fichiers d'éléments visuels - Ajout du fichier animations.json qui contient les ASCIIs à afficher dans la ligne de commande
- Ajout de la fonction gestion_exception qui permet de gérer une exception rencontrée durant le déroulement du script améliorant ainsi la lisibilité du code
- Amélioration de la condition qui vérifier que l'animation n'est pas déjà lancée lorsque on lance une nouvelle animation de chargement
- Ajout des tests unitaires pour la classe AfficheurCLI - Ajout du fichier .coveragerc pour exclure la vérification "if __name__ == "__main__"
- Déplacement du lancement des tests unitaires vers le fichier tests afin que le dossier de configuration de coverage.py soit pris en compte
- Modification de chemin dans test.yaml qui était mauvais depuis le déplacement des tests vers le dossier tests
- Modification du chemin de l'artefact qui était mauvais depuis le changement du dossier où les tests unitaires sont activés
- Modification d'une chaine de caractere qui était incorrecte pour les versions de python inférieur à 3.12
- Modification du chemin de l'artefact qui était incorretc depuis la dernière correction
- Modification de la branche de l'action des tests unitaires vers la branche develop
- Amélioration de la documentation des tests unitaires de test_afficheur_cli.py et test_main.py
- Ajout de la documentation pour la classe AfficheurCLI - Modification de la documentation des constantes de la classe AfficheurCLI
- Ajout des types des constantes dans la documentation des constantes de AfficheurCLI
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.