-
Notifications
You must be signed in to change notification settings - Fork 0
[docs]: Oppratert README i rot #51
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ser veldig bra ut. Fant bare et par småting, kommenterte de fleste i git men nevner de her:))
- "GraphØL" i implemented so far
- ".env. local" skal være
.env.local - setningen "
npm run dev. The backend does not need..." mangler starten, "Then run" e.l. - "Further functionality is planned..." gjentar det som står i Planned Functionality rett under, så den kan kanskje fjernes
- kanskje legge til en setning øverst i About the Project som nevner byene, eks: "The dataset also includes more than 34,000 cities linked to their countries, so the solution can be tested on a larger dataset."
- veldig småpirk, men noen overskrifter har fet skrift, mens andre ikke. kanskje bare fjerne ekstra fet skrift på de det gjelder ettersom de får tykkere skrift automatisk når det er overskrifer?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bra jobba:) Se kommentarer for endringer for å få den godkjent:)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Alle kommentarene er fikset og det ser bra ut nå! Eneste er at format:check feiler på README-filen, så kjør gjerne npm run format en siste gang før merge. Bra jobba!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nydelig, veldig godkjent!:)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Perfekt nå!
Hva er endret?
Endringene inkluderer:
Type endring
Test endringene
npm run lintfor å sjekke kodekvalitet med ESLintnpm run format:checkfor å sjekke formatering med Prettiernpm run buildfor å kontrollere TypeScript og produksjonsbyggetSjekkliste
Closes #36