diff --git a/guider/nye-repo.md b/guider/nye-repo.md index 456b980..30ce51a 100644 --- a/guider/nye-repo.md +++ b/guider/nye-repo.md @@ -11,12 +11,9 @@ Vi har en [standard README](../README.template.md) man kan ta utgangspunkt i. I README må det også fremkomme et kontaktpunkt for koden/teamet. Dersom repoet er åpent *skal* det fremkomme en måte omverden kan ta kontakt på (epost, navn på person, osv). Som et tillegg kan man oppgi slack-kanal osv, men dette er selvfølgelig myntet på interne ansatte (som det står i readme-template). ## Codeowners -Her må teamet stå som eiere, på formatet: `* @navikt/navn-på-team`. Eget team opprettes da på GitHub via navikt-organisasjonen. Enkeltpersoner skal ikke stå som codeowners. +I CODEOWNERS-filen kan man oppføre Github-teamet som eiere av repoet. OBS: Det er ikke sikkert du ønsker å gjøre dette. Konsekvensen er at for hver pull-request som opprettes i repoet blir teamet assignet som code reviewer. Det fører til at det går epostnotifikasjoner til alle medlemmer i teamet for alle endringer på alle pull-requester. -Liste over tilgjengelige NAV-team på Github er her: -https://github.com/orgs/navikt/teams - -Det maskinlesbare navnet på teamet er det man ser i URL-en når man går inn på teamet sin Github-side. +Gjøres på formatet: `* @navikt/navn-på-team`. Enkeltpersoner skal ikke stå som codeowners. Liste over tilgjengelige NAV-team på Github er her:https://github.com/orgs/navikt/teams Det maskinlesbare navnet på teamet er det man ser i URL-en når man går inn på teamet sin Github-side. Eksempel `CODEOWNERS`-fil: @@ -24,6 +21,7 @@ Eksempel `CODEOWNERS`-fil: * @navikt/navn-på-team ``` + ## Sanering av gammel kode Om det er flytting av eksisterende kode må både [kode og Git historikk saneres](sikkerhetsvask.md)