HL7Norway / best-practice

Utvikling og dokumentasjon av beste praksis og veiledninger for bruk av HL7 FHIR i Norge
https://hl7norway.github.io/best-practice/
17 stars 5 forks source link

Godkjenning/code review: Sjekkliste for endringer og tillegg i nasjonale profiler (og basis) #41

Open rockphotog opened 3 years ago

rockphotog commented 3 years ago

Vi må vurdere krav om code review og godkjenning til master. Usikker om skal ligge direkte under metode eller denne sjekklisten: https://github.com/HL7Norway/best-practice/blob/master/docs/no-national-checklist.md

Typisk krav kan være at det skal være minst to stk som ikke er forfatteren. Usikker om vi skal sette krav til roller, som at minst én av de skal være en "aktiv forvalter", eller om det kan være hvem som helst som er aktive.

Det samme gjelder basisprofiler, og der kan man sette krav om at én av godkjennerne skal være i TSK, siden de forvalter basisprofiler.

rockphotog commented 3 years ago

Forslag ligger her: https://github.com/HL7Norway/best-practice/blob/no-national-checklist-code-review/docs/no-national-checklist.md @thomiz @oaassv