Lucian Mocanu
Lucian Mocanu
@jorgebodega Could you please review this PR? FYI, I left the formatting in package.json unchanged on purpose, but I can also fix it easily if need be.
Related issue: #2098
@cassiosp it's the first time i'm seeing that website, but those are really outdated docs (from many years ago). Check the official website for docs: http://typeorm.io You can for sure...
@cassiosp the best documentation is the TSDoc, but we're improving the official website so LLMs can process it :) You can implement `NamingStrategyInterface` or simply extend `DefaultNamingStrategy` :)
Nowadays `globalThis` is available on all major platforms. Honestly, I don't see a reason to keep the `getGlobalVariable` function.
Would be great to have this fixed for NodeJs Workers as well before merging the PR. Also, some cleanup would be nice, i.e. replacing `window`/`global` with `globalThis` in both the...
@coderabbitai full review
> > Do we want to test with Node 24 as well? > > > > Afair there's a dependency package issue that builds to the native code (some of...
@gioboa Thanks, wanted to extract the non-SAP stuff from PR #11520 so I can make that easier to review :)
Still an issue