Gör en PR (kmom03, kmom06 och kmom10)

Följ denna instruktionen för att göra en pull request (PR) för att lämna in uppgiften i kmom03 och kmom06.

Förutsättning

Du har uppfyllt kraven i uppgiften kmom03/kmom06 och har gjort en vanlig inlämning enligt Resultat och redovisning.

Ordlista

PR: Pull request
Submitted: Får din PR när de automatiska testerna gått igenom. Nu ska du bara vänta på att en lärare kollar på din inlämning.
Test-passed: Får din PR när de automatiska testerna gått igenom.
Test-failed: Får din PR när något gått fel i de automatiska testerna. Ladda ner testerna igen, fixa felet lokalt och pusha till GitHub. Kör igång de automatiska testerna igen under “Actions” och markera din sista commit. Välj “Re-run all jobs” och sen “Re-run jobs”.
Approved/G/1: Helt klar med en uppgift.
Needs improvement/Ux/0.49: En lärare har bedömt din inlämning men tycker att något behöver förbättras i din inlämning. Använd dig av label Re-submitted när du har gjort din uppdatering.
Re-submitted: Du sätter den som label på din PR efter att du har fixat den komplettering en lärare gav på din inlämning.
Submission not approved/U/0: En lärare har bedömt din Re-submitted men tycker inte att det är tillräckligt bra. Du behöver göra en ny inlämning till nästa rättningstillfälle.

Mitt eget arbete

Det du lämnar in är ditt eget arbete. Om du har tagit hjälp av extern kod, eller om du har kopierat någon mindre del av koden, så är det tydligt angivet i kommentarerna i koden eller i din redovisningstext.

I samband med att du gör din PR så intygar du att arbetet är ditt eget och uppfyller de regler som finns för “Fusk och medveten vilseledning”.

Pull request (PR)

Du skall nu göra din inlämning som en PR mot följande branch.

bth/submit/kmom03

Följ stegen nedan för att skapa en PR.

Gå till din kod på GitHub

Du gör detta med hjälp av GitHub. Se till att du har pushat din aktuella kodbas till GitHub och öppna webbläsaren mot din kod på GitHub.

Image description — Bild: Kodbas på GitHub.

Gå till Pull requests

Klicka på (1) “Pull requests” i navbaren för din kodbas och därefter (2) den gröna knappen till höger “New pull request”.

Förbered att skapa en ny PR

Så här ser det ut när du klickat på “New Pull Request” (se föregående bild). Börja med (1) att välja “base” för din PR, där (2) väljer du bth/submit/kmom0<X> och vi låter “compare” vara main. Nu dyker det upp filer att jämföra och då (3) klickar du på den gröna knappen “Create pull request”.

Det här betyder att du vill merga main till bth/submit/kmom0<X>.

Öppna en PR mot en branch

Nu är du snart redo att skapa en PR. Börja (1) med att fylla i en titel på din PR, till exempel “Inlämning av kmom03” och lägg in dig själv som “Assignees”. Nu är det dags, öppna till din PR genom att klicka på (2) den gröna knappen “Create pull request”.

Nu är din PR öppnad.

En ny PR är öppnad

När det är klart kan det se ut så här.

Vänta ett par minuter, skrolla sen ner för att se att testerna har godkänts.

Testerna gick fel

Om något gick fel med testerna ser det ut så här i din PR. Du får en kommentar om vad som gick fel samt din PR får labeln tests-failed. I denna bilden har testerna för kmom01 och kmom02 gått bra men däremot har testerna för kmom03 gått fel.

Vanliga fel:

du har inte de nyaste testerna
du har inte den nyaste workflow filen

Nu behöver du kolla vad som gick fel, fixa felet och ladda upp koden igen. Enklast gör du det genom att:

Ladda ner testerna för det eller de kmom som gick fel och kör dem lokalt
Rätta felet och testkör lokalt.
Pusha din rättning till GitHub och testerna ska starta automatiskt.

Om testerna inte skulle starta automatiskt så går i under “Actions”, klicka på din senaste “action” och klicka på knappen “Re-run all jobs”. Därefter får du klicka på den gröna knappen “Re-run jobs” och så kör testerna igång igen ett en liten stund. Men detta ska inte behövas!

Testerna gick bra

Vänta ett par minuter för att se att testerna har godkänts. Då har din PR fått lablarna submitted och tests-passed.

Skrolla längre ner och du ska ha fått en kommentar om att testerna gick bra samt meddelandet “All checks have passed”.

PR kmom06

I inlämningen för kmom06 är det lite speciellt eftersom det är tester som ni gjort själva som ska köras. Dessutom körs ett antal tester för att kontrollera att inlämningen följer kraven.

Återkoppling på testerna får ni längst ner i PR för kmom06. Det kan se ut så här:

Student test result
Först körs era egna tester och resultatet blir antingen “student tests kmom05 ok” eller “FAIL:kmom05 student tests”.

Test result
Därefter körs testerna för att kolla om kraven följs. Eftersom testerna är hemliga så får ni en tabell med alla testfall, en tabell för kmom05 och en tabell för kmom06. Antingen står det en ✅ och då har testet gått igenom, eller så står det ett felmeddelande från testet.

Test results kmom05
Tabell med alla testfall för kmom05 med resultat.

Test results kmom06
Tabell med alla testfall för kmom06 med resultat.

PR kmom07/10

Du gör PR för projektet på samma sätt som för kmom03 och kmom06. Här finns inga tester utan du förväntas att själv se till att du följer kraven.

Feedback och bedömning

När läraren rättar din PR så får du återkoppling i kommentarer till din PR.

Approved / G

Om läraren tycker att allt ser bra ut får du en kommentar, label Approved och G i Canvas. Läraren mergar din PR och stänger den när den är godkänd. Du kommer också se vilken lärare som har varit “reviewer”. Du kan jobba vidare med nästa kmom.

Needs improvement / Ux

Om läraren tycker att något är fel som du behöver fixa får du label Needs improvement och Ux på Canvas. Det kommer finnas kommentarer på specifika rader. Om kommentaren innehåller en Warning som på bilden nedanför, då måste du åtgärda det i din kod. Om kommentaren inte innehåller Warning behöver du inte åtgärda det som står där.

När du har åtgärdat felen gör följande:

I din PR sätt label Re-submitted.
Pusha din kod til GitHub. Då körs testerna igen.
Om din fix funkade ska du nu få lablarna submitted och tests-passed.

Submission not approved / U

Om åtgärderna du gjorde i Re-submitted bedöms att inte vara tillräckliga får du label Submission not approved och U i Canvas. Det kommer finnas kommentarer som förklara felet och läraren kommer stänga din PR.

I kmom10, vid större fel, ges inte komplettering. Då kan du få U direkt.

Din inlämning kommer inte rättas mer. Din PR stängs. Du behöver göra en ny PR till nästa rättningstillfälle. I studieplanen hittar du när nästa rättningstillfälle är.