Skip to content

Commit

Permalink
feat!: Remove metrics API from the JS SDK (getsentry#14745)
Browse files Browse the repository at this point in the history
1. Removes `metrics` export and all it's usages
2. Removes all usages of `_metrics_summary` from the SDK
3. Removes `metricsShim` usage in the browser SDK CDN bundles
4. Removes all metrics related types from the SDK
  • Loading branch information
AbhiPrasad authored Dec 17, 2024
1 parent a921440 commit d773cb7
Show file tree
Hide file tree
Showing 69 changed files with 7 additions and 2,261 deletions.

This file was deleted.

This file was deleted.

This file was deleted.

This file was deleted.

This file was deleted.

159 changes: 0 additions & 159 deletions dev-packages/browser-integration-tests/suites/metrics/timing/test.ts

This file was deleted.

12 changes: 0 additions & 12 deletions dev-packages/browser-integration-tests/utils/helpers.ts
Original file line number Diff line number Diff line change
Expand Up @@ -270,18 +270,6 @@ export function shouldSkipFeedbackTest(): boolean {
return false;
}

/**
* We can only test metrics tests in certain bundles/packages:
* - NPM (ESM, CJS)
* - CDN bundles that include tracing
*
* @returns `true` if we should skip the metrics test
*/
export function shouldSkipMetricsTest(): boolean {
const bundle = process.env.PW_BUNDLE as string | undefined;
return bundle != null && !bundle.includes('tracing') && !bundle.includes('esm') && !bundle.includes('cjs');
}

/**
* We only test feature flags integrations in certain bundles/packages:
* - NPM (ESM, CJS)
Expand Down

This file was deleted.

18 changes: 0 additions & 18 deletions dev-packages/node-integration-tests/suites/metrics/should-exit.js

This file was deleted.

Loading

0 comments on commit d773cb7

Please sign in to comment.