Github action to check formatting

[LilleAila]

Github action som sjekker at koden er formatert før man får lov til å merge den. Dette vil sannsynligvis skape noen små problemer med PRs som allerede er åpen, men bør kunne fikses ved å git fetch origin && git merge origin/main, og kanskje fikse noen få conflicts. Alle bør kjøre npm run format på koden sin før commit.

TODO:

• [x] Lage en .env fil (github actions supporter secrets)
  • Egen admin account i backend for actions?
• [ ] Generere pocketbase types (bruke makefilen?)
• [x] Format all koden i repoen

npm run check er kommentert ut for nå, fordi den klager på at vi ikke har satt opp pocketbase types riktig enda

[kluvin]

1. ja

Nå er pocketbase på min private fly.io konto. Hvis vi skal ha en team-løsning er Google Cloud det eneste alternativet.

Jeg ville likt å konfigurert en enkel måte å deploye frontend til Netlify. Det er en annen tjeneste, men veldig optimalt for akkuratt dette.

Google Oauth er gjennom et prosjekt jeg laget på min private Google Cloud konto.

Vi bør ta hensyn til dette. @LilleAila vil du ha frontend på din konto, eller skal jeg ta det på min?

Vi kan lage en email tilknyttet prosjektet og ikke oss på sikt.

[LilleAila]

1. ja

Nå er pocketbase på min private fly.io konto. Hvis vi skal ha en team-løsning er Google Cloud det eneste alternativet.

Jeg ville likt å konfigurert en enkel måte å deploye frontend til Netlify. Det er en annen tjeneste, men veldig optimalt for akkuratt dette.

Google Oauth er gjennom et prosjekt jeg laget på min private Google Cloud konto.

Vi bør ta hensyn til dette. @LilleAila vil du ha frontend på din konto, eller skal jeg ta det på min?

Vi kan lage en email tilknyttet prosjektet og ikke oss på sikt.

Jeg lurte på om vi skulle bruke en raspberry pi på skolen til frontenden (og/eller pocketbase?), heller enn å bruke en ekstern tjeneste, men hvis vi bruker Netlify spiller det ikke så mye rolle, det går greit å ha det på min. Vi kunne kanskje skaffet en kaffediem@asvg.no?

[kluvin]

Tror vi kan self-hoste på en Pi. Jeg tenker kortsiktig her. Kan snakke om det på sikt, @LilleAila du kan hoste frontend, så lenge en github action er laget som autopubliserer. Trur det var mulig uten å måtte dele kredentialene dine med resten av oss.

[LilleAila]

npm run check virker ikke enda, det er på grunn av pocketbase types. Det burde kanskje gjøres i en annen PR

[kluvin]

@LilleAila det er sikkert ikke lov med http calls i github actions. types er ikke nødvendig under runtime, faktisk har jo ikke node noen kunnskap om dem i det hele tatt.

Det kan trygt fjernes.

[LilleAila]

@LilleAila det er sikkert ikke lov med http calls i github actions. types er ikke nødvendig under runtime, faktisk har jo ikke node noen kunnskap om dem i det hele tatt.

Det kan trygt fjernes.

npm run check sjekker koden, og klager på types som for eksempel drinks, da denne er satt til any. Hvis vi setter dette til en type fra en generert funksjon vil da actionen ikke kunne sjekke dette, siden filen ikke er med. HTTP calls bør være lov i actions, ellers hadde jo ikke steget for å installere dependencies fungert.