Skip to content

Code review

Ken Bonny requested to merge KenBonny/string-manipulation-kata:master into master

Hey Kris,

Jou code ziet er goed uit. Ik heb vooral de solution in een traditionele .net project structuur gegoten en de files verdeeld over 2 projecten: StringKata met de "product code" en StringKataTests met de test code.

Daarnaast zijn er nog een paar naming conventions die in .net gangbaar zijn (functies beginnen met een hoofdletter en private fields beginnen met _). De enige change aan de code zelf is dat als de generische code de types kan afleiden uit het gebruik van een methode, dan moeten de types niet meegegeven worden. Wat dit wil zeggen is dat je Select kan vervangen door Select, zie de class Different voor een voorbeeld.

Alle changes zitten in de eerste commit, ja al die veranderingen in een commit. Foei, stoute .net dev. ;) Daarna had ik nog even een probleempje met de ci configuratie. Zou nu ook opgelost moeten zijn, maar ik weet niet precies waar ik de ci builds kan checken, ik zag de mail die ik kreeg. :)

Merge request reports