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

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

Closed
eirslett opened this issue Jun 18, 2018 · 5 comments

Comments

@eirslett
Copy link

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 LICENSE.md-fil

Alle kodebasene til NAV skal (i utgangspunktet) lisensieres med MIT-lisens.
Bruk den lisensen her: https://github.com/navikt/utvikling/blob/master/LISENSIERING.md

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.

@eirslett
Copy link
Author

Minner om denne saken, som fortsatt ikke er løst. Hei @trulsjor, i følge robotanalysen var det du som gjorde siste commit i denne kodebasen, for 9 måneder siden. Kan du følge opp denne, eller tagge noen andre som har eierskap til koden?

@eirslett
Copy link
Author

eirslett commented Aug 6, 2018

Minner om denne saken, som fortsatt ikke er løst. Hei @trulsjor, i følge robotanalysen var det du som gjorde siste commit i denne kodebasen, for en måned siden. Kan du følge opp denne, eller tagge noen andre som har eierskap til koden?

@eirslett
Copy link
Author

Minner om denne saken, som fortsatt ikke er løst. Hei @trulsjor, i følge robotanalysen var det du som gjorde siste commit i denne kodebasen, for 2 måneder siden. Kan du følge opp denne, eller tagge noen andre som har eierskap til koden?

@eirslett
Copy link
Author

Minner om denne saka, som framleis ikkje er løyst. Hei @trulsjor, i fylgje robotanalysa var det du som gjorde siste commit i denne kodebasen, for 3 måneder sidan. Kan du fylgje opp denne, eller tagge nokon andre som har eigarskap til koden?

@eirslett
Copy link
Author

eirslett commented Oct 1, 2018

Minner om denne saka, som framleis ikkje er løyst. Hei @trulsjor, i fylgje robotanalysa var det du som gjorde siste commit i denne kodebasen, for 3 månader sidan. Kan du fylgje opp denne, eller tagge nokon andre som har eigarskap til koden?

@eirslett eirslett closed this as completed Oct 1, 2018
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

No branches or pull requests

1 participant