Feedback #10
Description
Titta på hur repot "mår", hur ligger de till i sin planering? Hur ofta commitar dem?
Gruppen commitar relativt regelnbundet vilket är bra. När det kommer till själva huvudspelet är de enligt antalet utförda issues ca 80% klara. Det dem har kvar är att "calculate and color area".
Hur väl har de planerat?
Det finns två milestones. Att skapa själva spelet och att sedan göra om det till multiplayer. I varje milestones finns också de issues som ska genomföras. Så planeringen verkar strukturerad och bra. Självklart hade det kunnat vara fler milestones som snarare är uppdelade tidsmässigt (vecka1, vecka2...) men detta anser vi är ett personligt val som har med gruppens organisationstänk att göra.
Verkar det som alla har bidragit eller kommer alla stora commits från samma person?
Antalet commits ligger på 4 och 6. Detta är relativt jämnt, men beror av storleken på commits. Det finns också möjlighet att kodandet skett gemensamt varav båda arbetat och commitat från samma dator.
Har de några gamla branches som ligger kvar och blir svåra att merga in?
De enda branches som finns just nu är en varsin i slutet av processen. Alltså, de har båda skapat en branch från samma version av main. Detta borde innebära att problem inte kommer att uppstå under merge om inte båda kodar i samma område.
När gjordes senaste commiten, är det längesedan? Har de tydliga commit-meddelanden?
Senaste commiten var idag 13 maj. Meddelandena är mestadels beskrivande och tydliga. Ibland förekommer "Updated" vilket vi inte riktigt vet vad det innebär. Där hade man kunnat vara mer detaljerad och beskriva vad som faktiskt ändrats. På ett ställe förekom samma kommentar två gånger efter varandra, där man kanske borde beskriva vad skillnaden är mellan dem.
Hur är koden strukturerad, är det tydligt? (Behöver inte gå jättedetaljerat på detta bara överskådligt.)
Koden är uppdelad i funktioner och har kommentarer som beskriver de flesta av dem. Däremot finns inslag av kommentarer som är lite svårförståeliga, däribland "bruh" ;). Vissa funktioner hade kunnat delas upp i filer om möjligt, då koden blir väldigt lång och det blir svårt att leta sig fram till det man söker. Annars ser det bra ut!
Något övrigt ni vill kommentera?
Ska bli kul att se resultatet!
Saga
Kalle