Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ny modul om kodelister i introkurset #1722

Merged
merged 6 commits into from
Aug 2, 2024
Merged

Ny modul om kodelister i introkurset #1722

merged 6 commits into from
Aug 2, 2024

Conversation

TomasEng
Copy link
Contributor

@TomasEng TomasEng commented Jul 24, 2024

Har lagt til en modul om kodelister i det oppdaterte introkurset, i tillegg til å ta en liten språkvask i eksisterende moduler og kodelisteguiden.

@TomasEng TomasEng force-pushed the introkurs-ny-modul branch 3 times, most recently from e514b4d to 8b310c0 Compare July 25, 2024 06:59
@TomasEng TomasEng linked an issue Jul 25, 2024 that may be closed by this pull request
@TomasEng TomasEng linked an issue Jul 25, 2024 that may be closed by this pull request
@TomasEng TomasEng marked this pull request as ready for review July 25, 2024 07:04
@TomasEng TomasEng force-pushed the introkurs-ny-modul branch from 8b310c0 to 4ab8388 Compare July 25, 2024 07:13
@TomasEng TomasEng force-pushed the introkurs-ny-modul branch from 4ab8388 to 4bdecab Compare July 25, 2024 07:34
Copy link
Contributor

@Ildest Ildest left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Da er jeg ferdig med gjennomgangen @TomasEng. Det er ganske mye, siden jeg også har revidert det som ikke handlet om kodelister. Noen punkter å ta hensyn til:

  • Bruk "du" ikke "man" i instrukspunkter. I generelle deler kan vi bruke "vi", slik det er brukt noen steder her.
  • Ha ett punkt (ett gjøremål) per instrukslinje.
  • Jeg foreslår å ha overskriften unummerert og heller ha flere mellomoverskrifter for å unngå lange instrukser. En prosedyre bør ikke ha mer enn fem punkter, maks sju. Vi har prosedyrer i vårt introkurs med over 20 punkter, da mister vi leseren tidlig, de må ofte refokusere midt i for å huske hvor langt de har kommet.
  • I overskriften rett før en trinnvis veiledning, bruk verbet i imperativ ("Legg til").
  • I generelle overskrifter før forklarende tekst om et emne, bruk infinitiv av verbet ("Legge til").
  • Når det gjelder denne revisjonen. - det ble mye fordi jeg tok tak i alt som sto her: Hvis du vil kan du oppdatere kun det som står om kodelister, og så kan vi overlate resten til en ny sak, og jeg kan oppdatere det som gjelder resten av kurset? Litt opp til deg, men forstår om du ikke har tid til å ta tak i alt dette.
  • Fint om du leser korrektur på mine forslag når du går gjennom, det kan alltid være noen stavefeil, så da blir det en ekstra kvalitetssikring.
  • Hvis du kan publisere etter at du har endret (har vi en testserver for publisering av dok?), tar jeg gjerne en test av det å legge til kodelister igjen, for å se at det fungerer slik det ble revidert.

Copy link
Contributor

@Ildest Ildest left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nå tror jeg alle kommentarene, unntatt et par der jeg svarte deg, er løst.

@TomasEng TomasEng merged commit 4eac0e4 into master Aug 2, 2024
3 checks passed
@TomasEng TomasEng deleted the introkurs-ny-modul branch August 2, 2024 12:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Module 6: Options and static code lists
2 participants