realm / realm-js

Realm is a mobile database: an alternative to SQLite & key-value stores
https://realm.io
Apache License 2.0
5.62k stars 558 forks source link

Bundling with Metro warns with "Require cycles are allowed, but can result in uninitialized values." #6565

Open kraenhansen opened 1 month ago

kraenhansen commented 1 month ago

We used to resolve these require cycles as part of producing a rollup bundle, but these will now surface as warnings when bundling with Metro.

We could probably revisit the decision to use a internal.ts to control the import order and instead focus on breaking the cycles:

https://github.com/realm/realm-js/blob/1e1e48a0f906d471b4a046168ed8cd628ef2d70f/packages/realm/src/internal.ts#L21

sync-by-unito[bot] commented 1 month ago

➤ PM Bot commented:

Jira ticket: RJS-2769