-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
feat(core): Deprecate logger in favor of debug #17040
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
MIGRATION.md
Outdated
|
||
## Deprecated `@sentry/core` SDK internal `logger` export | ||
|
||
The `logger` export from `@sentry/core` has been deprecated in favor of the `debug` export. `debug` only exposes `log`, `warn`, and `error` methods but is otherwise identical to `logger`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we mention that this is about the internal logger and unrelated to the Logs product?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah good call, done with 4064ff1
Co-authored-by: Francesco Gringl-Novy <francesco.novy@sentry.io>
size-limit report 📦
|
resolves #16905
also renames the file to be
debug-logger.ts
instead oflogger.ts
to make usage inside@sentry/core
a little easier to understand.Added a note to
MIGRATION.md
, but let me know if I should remove that.