21 config klargjør første innlevering #29
Conversation
Co-authored-by: tordtst <tordtst@ntnu.no>
Co-authored-by: birkjr <birkjr@ntnu.no>
Co-authored-by: andretg <andretg@ntnu.no> Co-authored-by: tordtst <tordtst@ntnu.no>
Co-authored-by: birkjr <birkjr@ntnu.no> Co-authored-by: andretg <andretg@ntnu.no>
Co-authored-by: tordtst <tordtst@ntnu.no>
There was a problem hiding this comment.
Ser bra ut, godkjent
Jeg testet branchen i en ren kopi med npm ci. Typecheck, lint, format:check, test (79 av 79) og build går alle uten feil. Jeg sjekket også katalogen, detaljvisningen og utleieskjemaet i 320, 360, 768 og 1280 px. Ingen av dem gir horisontal scrolling, og konsollen har ingen feil.
Det som er bra:
Avsnittene om data, planen videre og «Dette ønsker vi tilbakemelding på» gjør det mye lettere for medstudentene å forstå idéen og gi tilbakemelding.
engines og README sier nå det samme om hvilken Node-versjon som trengs.
Favicon er på plass, og den utdaterte merknaden om at den manglet er fjernet fra testing.md.
Liten ting, ikke nødvendig for denne PR-en:
Med Node 24.13 skriver npm ci en EBADENGINE-advarsel, men alt kjører. Det er som forventet etter det nye kravet.
No description provided.