Upgrade Jest to 30.2

[SjaakSchilperoort]

Resolves: https://github.com/observation/app/issues/1111

[jonathan-waarneming-nl]

👌

Als ik yarn remove @types/jest jest ts-jest && yarn add -D @types/jest jest ts-jest run heb ik nog wat changes in de yarn.lock. Is dat erg?

[SjaakSchilperoort]

Waarom zou je dat doen? Krijg je dan ook nieuwere versies van deze dependencies? Of is enkel de formattering van yarn.lock veranderd?

[jonathan-waarneming-nl]

Waarom zou je dat doen? Krijg je dan ook nieuwere versies van deze dependencies? Of is enkel de formattering van yarn.lock veranderd?

Volgens mij gooit ie verouderde/ongebruikte packages weg en herformateert ie wat dingen.

Maar waarom zou je 't doen? Het voelt up-to-dater, maar dit is niet echt onderbouwd. Het is een nieuwe major version, dus dan mag een clean slate?

[SjaakSchilperoort]

Maar waarom zou je 't doen? Het voelt up-to-dater, maar dit is niet echt onderbouwd. Het is een nieuwe major version, dus dan mag een clean slate?

Dat kan dan ook rigoureuzer: na een upgrade van React Native plus dependencies kunnen we yarn.lock vanaf scratch hergenereren. Even gecheckt, node_modules wordt 43% kleiner. Dat maakt niet uit voor de installed app size (tree shaking verwijdert alle ongebruikte code), maar de CI wordt wel sneller. De andere repo's: node_modules voor Observation wordt 12% kleiner, ObsIdentify 19%.

Wel een risico: we kunnen onbedoelde upgrades van transient dependencies krijgen, en dat zou problemen kunnen veroorzaken. Dat risico lijkt me niet heel groot.

[jonathan-waarneming-nl]

Klinkt als een goed plan! ;)

[SjaakSchilperoort]

Moeten we wel separaat doen. Kunnen we komende maandag naar kijken, tezamen met de ideale TypeScript-configuratie. En misschien ook eens naar yarn 4 kijken.