Produksjonsprosedyre

Ting som bør være på plass før man merger en branch inn i master, altså etter man har en pull request. Dette skal være brukertester.

Sjekkliste:

• Teste om spillet funker som det skal.
  • Roboten beveger seg som den skal
    • Skal eksempelvis ikke kunne gå gjennom vegger.
    • Option-kort utfører bevegelsen som er forventet
    • Rullebåndene skal skyve deg bortover om du havner der. Avstand avhenger av hvilket bånd
    • Du skal kunne dytte en annen spiller, kun hvis du står rotert med fronten mot den andre spilleren
    • Laser skal gjøre skade, varierer fra laser til laser hvor mye
  • Spillet kan eksempelvis kjøre en sekvens uten å kræsje (Trenger nødvendigvis ikke kjører flere)
  • To personer kan gjerne teste (sjekke at det funker for begge)
• Sørge for at branchen er à jour med master.
• Det bør være tester som dekker koden man legger til.
• Passerer builden? Dette skal man kunne se fra forsiden av gruppeprosjektet under status yellowbots
• Har du fulgt kodestilen som gruppen på forhånd har blitt enig om? Her vil codacy sannsyligvis komme med flere forbedringspunkter
• Sørg for at en eventuell pull request har en beskrivende tittel og beskrivelse, slik at det blir mer oversiktlig å forstå hva som har blitt lagt til.
• Husk å oppdatere koden med kommentarer slik at det blir enklere for andre i gruppa å forstå hva du har gjort.
• Sjekke at .jar-filen produseres korrekt.
• Korrigere det codacy klager på, i en evt. pull request.

Sjekklisten er viktig og bør følges.