opral / inlang-paraglide-js

Tree-shakable i18n library build on the inlang ecosystem.
https://inlang.com/m/gerre34r/library-inlang-paraglideJs
52 stars 1 forks source link

Circular dependecy in svelte@5.0.0-next.143 in CI #157

Closed jldec closed 5 months ago

jldec commented 5 months ago

Context

I don't know if this is serious, but many of the CI runs in GitHub are showing messages like this. I went back a few weeks, and it appears to have been happening for while.

Circular dependency: ../../../../node_modules/.pnpm/svelte@5.0.0-next.143/node_modules/svelte/src/internal/client/runtime.js -> ../../../../node_modules/.pnpm/svelte@5.0.0-next.143/node_modules/svelte/src/internal/client/proxy.js -> ../../../../node_modules/.pnpm/svelte@5.0.0-next.143/node_modules/svelte/src/internal/client/dev/ownership.js -> ../../../../node_modules/.pnpm/svelte@5.0.0-next.143/node_modules/svelte/src/internal/client/runtime.js

To see these, open an instance of the Continuous Integration workflow like this one and use the built-in search to look for "Circular"

Screenshot 2024-06-18 at 16 58 41

Proposal

decide if worth investigating or close the issue if not 😃

LorisSigrist commented 5 months ago

This seems to be caused by how the integration tests bundle svelte. Not a problem, but I'll try to suppress the error

LorisSigrist commented 5 months ago

I supplied a custom onwarn handler that silences that specific warning, this reduces the noise in CI. Thanks for bringing it to my attention

jldec commented 5 months ago

Thanks @LorisSigrist