Skip to content

Comments

refactor: Refactorisation du code suite aux retours du débrief#1

Open
NilinC wants to merge 10 commits intomainfrom
refactoring
Open

refactor: Refactorisation du code suite aux retours du débrief#1
NilinC wants to merge 10 commits intomainfrom
refactoring

Conversation

@NilinC
Copy link
Owner

@NilinC NilinC commented Apr 5, 2023

Suite aux retours lors de l'entretien de débrief.

Voici les modifications suivantes :

  • On vérifie que le lien du fichier existe
  • On utilise un tableau avec des clés au lieu d'index pour récupérer les données
  • On déplace le formatage des données dans un service
  • Interpréter le \r dans la description
  • Formater le prix à 2 décimales après la virgule, arrondi au dixième
  • Ajouter un paramètre à la commande pour afficher un JSON plutôt qu'un tableau formaté
  • tâche CRON
  • Mettre à jour le README

@NilinC NilinC changed the title refactor: On vérifie que le lien du fichier existe refactor: Refactorisation du code suite aux retours du débrief Apr 7, 2023
NilinC added 4 commits April 14, 2023 16:12
…N au lieu d'un tableau formaté

* Définition d'une option pour encoder le résultat en JSON
* Création d'une entité Headers au lieu d'une simple définition en constante
* Réfactorisation de la création des rows pour la déplacer dans une nouvelle méthode
@NilinC NilinC force-pushed the refactoring branch 2 times, most recently from 55ed6cd to 8f52789 Compare April 17, 2023 13:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant