From 7fbd56baa458f0db052ca316c8248b9aefb1c864 Mon Sep 17 00:00:00 2001 From: Luca Forstner Date: Fri, 12 Jul 2024 13:16:38 +0000 Subject: [PATCH] Review feedback --- CHANGELOG.md | 3 +-- packages/core/src/utils/hasTracingEnabled.ts | 1 + packages/types/src/options.ts | 6 +++--- 3 files changed, 5 insertions(+), 5 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 3cb4e06bbddb..6fd671757241 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -15,8 +15,7 @@ The `enableTracing` option has been deprecated and will be removed in the next major version. We recommend removing it in favor of the `tracesSampleRate` and `tracesSampler` options. If you want to enable performance monitoring, please set the `tracesSampleRate` to a sample rate of your choice, or provide a sampling function as `tracesSampler` option -instead. If you wan't to disable performance monitoring, remove the `tracesSampler` option and set the -`tracesSampleRate` to `0` or `undefined`. +instead. If you wan't to disable performance monitoring, remove the `tracesSampler` and `tracesSampleRate` options. Work in this release was contributed by @GitSquared. Thank you for your contribution! diff --git a/packages/core/src/utils/hasTracingEnabled.ts b/packages/core/src/utils/hasTracingEnabled.ts index 97463d9d5e5e..5e673bc08caa 100644 --- a/packages/core/src/utils/hasTracingEnabled.ts +++ b/packages/core/src/utils/hasTracingEnabled.ts @@ -17,6 +17,7 @@ export function hasTracingEnabled( } const options = maybeOptions || getClientOptions(); + // eslint-disable-next-line deprecation/deprecation return !!options && (options.enableTracing || 'tracesSampleRate' in options || 'tracesSampler' in options); } diff --git a/packages/types/src/options.ts b/packages/types/src/options.ts index 82431461d3ba..d6c407d60bd0 100644 --- a/packages/types/src/options.ts +++ b/packages/types/src/options.ts @@ -93,9 +93,9 @@ export interface ClientOptions