GemeenteNijmegen / webformulieren-submissionstorage

Storage of submitted forms
European Union Public License 1.2
0 stars 0 forks source link

fix: attempt to build pipeline without testcontainers #247

Closed WietekeNijmegen closed 3 months ago

WietekeNijmegen commented 3 months ago

Deze versie bevat geen wrap-ansi-cjs. Deze werd gebruikt door test-containers en is drie weken geleden toegevoegd in een auto-upgrade. Lijkt wel de foutmelding te veroorzaken, aangezien de waarschuwing over wrap-ansi overwrites er drie weken geleden nog niet waren.

https://github.com/GemeenteNijmegen/webformulieren-submissionstorage/commit/f1dbc344b27b0c68649329a6c808bc935f8d492e Deze auto-upgrade was drie weken geleden, maar de build ging pas vandaag stuk. De waarschuwing

warning Pattern ["wrap-ansi@^7.0.0"] is trying to unpack in the same destination "/usr/local/share/.cache/yarn/v6/npm-wrap-ansi-cjs-7.0.0-67e145cff510a6a6984bdf1152911d69d2eb9e43-integrity/node_modules/wrap-ansi-cjs" as pattern ["wrap-ansi-cjs@npm:wrap-ansi@^7.0.0"]. This could result in non-deterministic behavior, skipping.

In combinatie met de error: Error [ERR_REQUIRE_ESM]: require() of ES Module /codebuild/output/src3832702998/src/node_modules/wrap-ansi/index.js from /codebuild/output/src3832702998/src/node_modules/jest-cli/node_modules/cliui/build/index.cjs not supported. Instead change the require of index.js in /codebuild/output/src3832702998/src/node_modules/jest-cli/node_modules/cliui/build/index.cjs to a dynamic import() which is available in all CommonJS modules.

Is dus pas vandaag verschenen. Dit volgt niet helemaal de logica van de update die drie weken geleden is gebeurd.

Maar de laatste geslaagde build in de nacht van dinsdag op woensdag 21 augustus bevat niet die warning over wrap ansi. Dus wellicht een poging waard?

joostvanderborg commented 3 months ago

De dependency update-build lijkt ook geslaagd. Deze even achter de hand houden voor als we daarop niet door kunnen.