Skip to content

Rework guilhem#9

Open
gdemiollis wants to merge 12 commits intoInsiteo:masterfrom
gdemiollis:Rework_guilhem
Open

Rework guilhem#9
gdemiollis wants to merge 12 commits intoInsiteo:masterfrom
gdemiollis:Rework_guilhem

Conversation

@gdemiollis
Copy link
Copy Markdown

Bonjour,

Afin de préparer mon entrevue avec Augustin MONSAINGEON, je suis passé à travers ce code .
J'ai eu un peu de mal à le comprendre donc je l'ai refactoré selon ma compréhension pour avoir de plus petites classes avec chacune sa responsabilité.
Étant donné que je ne connais pas toutes les subtilités de cette application, il se peut que j'ai introduit des bugs, donc ce pul request n'est pas à utiliser tel-quel.

J'aurais aimé pouvoir ajouter des tests unitaires mais le temps m'a manqué.

Pour résumé, cette branche représente beaucoup la façon dont je code et ce à quoi vous pourrez vous attendre si je rentre dans votre équipe.

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