makefile: move npm scripts

[IldenH]

Flytter alt av npm scripts in i makefilen for å være mer konsekvent og gjøre det lettere å bruke prosjektet.

[github-actions]

Visit the preview URL for this PR (updated for commit 08dfe50):

https://kaffe-diem--pr107-make-everything-ibtv0cze.web.app

_{(expires Sun, 19 Jan 2025 17:45:32 GMT)}

_{🔥 via Firebase Hosting GitHub Action 🌎}

_{Sign: 442774dd5e8e2b443bd55c0d98e4d4879d91742b}

[LilleAila]

Inneholder stringen som printes til terminalen secrets? I så fall er det nok best å skjule det.

[kluvin]

nitpick, eller protip, eller #LPT, eller noe slikt:

Når man kjører en pakke med npx trenger man ikke å ha pakken installert.

https://docs.npmjs.com/cli/v8/commands/npx

[LilleAila]

nitpick, eller protip, eller #LPT, eller noe slikt:

Når man kjører en pakke med npx trenger man ikke å ha pakken installert.

https://docs.npmjs.com/cli/v8/commands/npx

Ja, men alt er bevisst lagt til i devDependencies fordi det er mye mer declarative og locker dependencies sentralt til riktig versjon

[kluvin]

Ja, men alt er bevisst lagt til i devDependencies fordi det er mye mer declarative og locker dependencies sentralt til riktig versjon

Jeg synes absolutt det er nydelig, var bare litt forvirrende når jeg leste npx. Selv om npx først leter lokalt, tenker jeg det er lurt å være eksplisitt å bruke npm run.

[IldenH]

Dette er en diskusjon på om npm scripts versus makefile.
Det gir ikke mening å ha en makefile slik vi hadde hvor
make x = npm run x

Jeg valgte å bruke makefile til alt da:

• commanden er kortere
• env fil er lettere
• lettere å lese yaml enn package.json
• lettere å skjule commands som pb_types secrets

Npx er definitivt forrvirrende. Kan vurdere å bytte det til npm exec men det er samme tingen bare mer verbose.