feat(core)!: Remove hasTracingEnabled (#17072)

`hasTracingEnabled` was deprecated in v9, use `hasSpansEnabled` as a
direct replacement.

Author: andreiborza

MIGRATION.md

@@ -6,7 +6,10 @@ These docs walk through how to migrate our JavaScript SDKs through different maj
 - Upgrading from [SDK 6.x to 7.x](./docs/migration/v6-to-v7.md)
 - Upgrading from [SDK 7.x to 8.x](./docs/migration/v7-to-v8.md)
 - Upgrading from [SDK 8.x to 9.x](./docs/migration/v8-to-v9.md)
-- Upgrading from [SDK 9.x to 10.x](#upgrading-from-9x-to-10x)
+  <<<<<<< HEAD
+- # Upgrading from [SDK 9.x to 10.x](#upgrading-from-9x-to-10x)
+- Upgrading from [SDK 9.x to 10.x](#upgrading-from-8x-to-9x)
+  > > > > > > > f726d5ab5 (chore: Add migration guide for v10 and move v8->v9 guide to docs)
 
 # Upgrading from 9.x to 10.x
 
@@ -37,6 +40,7 @@ Updates and fixes for version 9 will be published as `SentryNodeServerlessSDKv9`
 ### `@sentry/core` / All SDKs
 
 - `BaseClient` was removed, use `Client` as a direct replacement.
+- `hasTracingEnabled` was removed, use `hasSpansEnabled` as a direct replacement.
 
 ## No Version Support Timeline
 

packages/core/src/index.ts

@@ -59,8 +59,6 @@ export { getIntegrationsToSetup, addIntegration, defineIntegration } from './int
 export { applyScopeDataToEvent, mergeScopeData } from './utils/applyScopeDataToEvent';
 export { prepareEvent } from './utils/prepareEvent';
 export { createCheckInEnvelope } from './checkin';
-// eslint-disable-next-line deprecation/deprecation
-export { hasTracingEnabled } from './utils/hasSpansEnabled';
 export { hasSpansEnabled } from './utils/hasSpansEnabled';
 export { isSentryRequestUrl } from './utils/isSentryRequestUrl';
 export { handleCallbackErrors } from './utils/handleCallbackErrors';

packages/core/src/utils/hasSpansEnabled.ts

@@ -34,11 +34,3 @@ export function hasSpansEnabled(
     (options.tracesSampleRate != null || !!options.tracesSampler)
   );
 }
-
-/**
- * @see JSDoc of `hasSpansEnabled`
- * @deprecated Use `hasSpansEnabled` instead, which is a more accurately named version of this function.
- * This function will be removed in the next major version of the SDK.
- */
-// TODO(v10): Remove this export
-export const hasTracingEnabled = hasSpansEnabled;

packages/core/test/lib/utils/hasTracingEnabled.test.ts renamed to packages/core/test/lib/utils/hasSpansEnabled.test.ts

@@ -1,5 +1,5 @@
 import { describe, expect, it } from 'vitest';
-import { hasSpansEnabled, hasTracingEnabled } from '../../../src';
+import { hasSpansEnabled } from '../../../src';
 
 describe('hasSpansEnabled', () => {
   const tracesSampler = () => 1;
@@ -15,7 +15,5 @@ describe('hasSpansEnabled', () => {
     ['With tracesSampler and tracesSampleRate', { tracesSampler, tracesSampleRate }, true],
   ])('%s', (_: string, input: Parameters<typeof hasSpansEnabled>[0], output: ReturnType<typeof hasSpansEnabled>) => {
     expect(hasSpansEnabled(input)).toBe(output);
-    // eslint-disable-next-line deprecation/deprecation
-    expect(hasTracingEnabled(input)).toBe(output);
   });
 });