Overdragelse og code review af afgangskomponenten

[SigneA-hm]

@hypesystem adgangskomponenten skal placeres i et selvstændigt repository her: https://github.com/os2ai
Der skal tilføres en MPL 2.0 licens, som er OS2-standard.

@lasseborly eller @cableman udfører code review. Vil I melde ind, om der er én af jer der kan gøre det inden for den næste uge?

Vi har ikke en ramme for code review endnu. Derfor indfører vi et basis-review, som er vores fælles sikkerhedsnet. Formålet er at sikre, at koden er læsbar for hele teamet, og at vi ikke overser kritiske logiske fejl, før vi merger. Vi holder det let og fokuseret på 'fire øjne-princippet', indtil vi har besluttet os for en mere formel ramme.

[SigneA-hm]

@lasseborly har allerede delvist lavet codereview i udviklingsprocessen, hvorfor @lasseborly fortsætter.

Komponenten skal med i næste release, hvordan gør vi dét nu når det er AarhusAI, der bygger releases @cableman?

[hypesystem]

Jeg lægger koden her https://github.com/os2ai/adgangskomponent og laver efterfølgende PR på udgivelse af helm chart og docker images via Github Actions

[lasseborly]

@SigneA-hm jeg er færdig med første runde af code review.

[SigneA-hm]

så hvad så nu @lasseborly og @hypesystem ?

[lasseborly]

Jeg tænker at @hypesystem går mit review igennem når han får tid og vurderer om der er noget der skal laves om. Ellers har jeg godkendt det, da der ikke er noget jeg lige kan se ikke kan tilrettes eller klarlægges på senere tidspunkt.