navikt/storebror

Storebror passer på å holde Git-repositories oppdatert med korrekt konfigurasjon.

storebror

Storebror passer på å holde Git-repositories oppdatert med korrekt konfigurasjon.

WORK IN PROGRESS

Akkurat nå sjekkes det om nais-manifester i et repository inneholder en seksjon med prometheus som er skrudd på. Hvis ikke, setter Storebror den inn i kildekoden, og gjør en git commit.

Utviklingsmiljø

• Golang >= 1.10
• Dep

dep ensure
go build
./storebror https://github.com/path/to/repository
# eller:
./storebror file:///path/to/repository

Docker-container

docker build -t navikt/storebror .  # eller "make"
docker run --rm -it navikt/storebror https://github.com/path/to/repository

storebror's Issues

Sjekke om manifest.json fra create-react-app

Create-react-app generer en default manifest.json fil for appen. Denne har en del placeholdere som må endres. (f.eks. bytte navn på applikasjonen). Det hadde vært fint om storebror kunne ha verifisert at man husket å endre denne filen.

Eksempel: https://github.com/navikt/arbeidssokerregistrering/blob/b7daa0a9f06e02a6924bcdda6fb7debda25b4838/public/manifest.json

Rette feil/mangler i henhold til krav i navikt/utvikling (funnet av roboten repo-linter)

Dette er en autogenerert issue, laget av et skript som går gjennom alle NAV sine kodebaser på Github og gjør diverse sjekker. Her er en liste over ting som må endres.

Kodebasen mangler en CODEOWNERS-fil

Dette er en fil som skal ligge i rotkatalogen, og angir hvilket team som eier kodebasen, på et maskinlesbart format. Den enkleste varianten, som vil holde for de fleste, er å ha en CODEOWNERS-fil som ser slik ut: (Merk at det skal være asterisk/stjerne foran navn på teamet!)

* @navikt/teamnavn

Gyldige teamnavn på Github er:

• @navikt/apen-kildekode (Åpen kildekode)
• @navikt/aura (aura)
• @navikt/bris (bris)
• @navikt/core (Core)
• @navikt/dagpenger (Dagpenger)
• @navikt/developers (Developers)
• @navikt/digisos (DIGISOS)
• @navikt/digisyfo (Digisyfo)
• @navikt/dolly-freg (Dolly-freg)
• @navikt/forenklet-oppfolging (forenklet oppfølging)
• @navikt/gjenlevende (gjenlevende)
• @navikt/gosys (Gosys)
• @navikt/informasjon-og-veiledning (Informasjon og veiledning)
• @navikt/integrasjon (Integrasjon)
• @navikt/kuhr-helse (kuhr-helse)
• @navikt/machine-users (machine-users)
• @navikt/melosys (Melosys)
• @navikt/mobilarbeidsplass (MobilArbeidsplass)
• @navikt/nav-frontend-admins (nav-frontend-admins)
• @navikt/nav-labs (NAV-labs)
• @navikt/pam (PAM)
• @navikt/peon (peon)
• @navikt/personoversikt (Personoversikt)
• @navikt/pus (pus)
• @navikt/redaktorer (redaktører)
• @navikt/samordning (samordning)
• @navikt/selvbetjening-core (selvbetjening-core)
• @navikt/team-arbeid (Team Arbeid)
• @navikt/team-oppfolging-fo (Team oppfølging - FO)
• @navikt/team-personbruker (Team Personbruker)
• @navikt/team-soknad (Team søknad)

Her er en oppdatert liste over team i Github.
Mangler teamet deres i lista? Ta kontakt med noen i Core-teamet på Github, så kan de opprette et team til dere.

Hvis det trengs spesielle tilpasninger, så ligger det mer dokumentasjon om CODEOWNERS-filformatet her:

https://help.github.com/articles/about-codeowners/

Spørsmål og svar

Jeg har meninger om disse rådene - kan jeg komme med tilbakemeldinger?

Skriv i vei, på Slack-kanalen #open-source.

Kodebasen vår er ikke open source, derfor er det ikke noe poeng

Selv om koden i dag ikke er åpen for innsyn, så ta høyde for at den kan komme til å bli det i fremtiden. Uansett så vil forbedringene være til hjelp, enten kodebasen er åpen eller ei!

Hvem har ansvaret for å fikse det her?

Det er i utgangspunktet den/de/teamet som eier kodebasen som må fikse.

Det er en feil i rådene

Alle roboter gjør jo feil, denne også. Lag en issue på https://github.com/navikt/repo-linter.