Skip to content

feat: Deprecate registerEsmLoaderHooks.include and registerEsmLoaderHooks.exclude#14486

Merged
lforst merged 3 commits intodevelopfrom
lforst-registerEsmLoaderHooks-depr
Nov 27, 2024
Merged

feat: Deprecate registerEsmLoaderHooks.include and registerEsmLoaderHooks.exclude#14486
lforst merged 3 commits intodevelopfrom
lforst-registerEsmLoaderHooks-depr

Conversation

@lforst
Copy link
Copy Markdown
Contributor

@lforst lforst commented Nov 26, 2024

Ref: #14483

Deprecates registerEsmLoaderHooks.include and registerEsmLoaderHooks.exclude and adds notes to all EsmLoaderHookOptions what it takes towards migrating to the new default of making onlyIncludeInstrumentedModules the default behavior.

});

const esmWarning =
'[Sentry] You are using Node.js in ESM mode ("import syntax"). The Sentry Node.js SDK is not compatible with ESM in Node.js versions before 18.19.0 or before 20.6.0. Please either build your application with CommonJS ("require() syntax"), or use version 7.x of the Sentry Node.js SDK.';
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Snuck this in. It's probably not in our interest to point people towards v7 any longer.

@lforst lforst requested review from mydea and timfish November 27, 2024 07:56
@lforst lforst merged commit 3e7969f into develop Nov 27, 2024
@lforst lforst deleted the lforst-registerEsmLoaderHooks-depr branch November 27, 2024 09:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants