Skip to content

Commit

Permalink
feat: Remove @sentry/opentelemetry-node package (#10906)
Browse files Browse the repository at this point in the history
Users can/should instead just use `@sentry/node`, or use
`@sentry/opentelemetry` themselves.

Closes #9843
  • Loading branch information
mydea authored Mar 4, 2024
1 parent 09f2a69 commit ef6e6e5
Show file tree
Hide file tree
Showing 32 changed files with 2 additions and 2,569 deletions.
5 changes: 0 additions & 5 deletions .craft.yml
Original file line number Diff line number Diff line change
Expand Up @@ -91,9 +91,6 @@ targets:
- name: npm
id: '@sentry/serverless'
includeNames: /^sentry-serverless-\d.*\.tgz$/
- name: npm
id: '@sentry/opentelemetry-node'
includeNames: /^sentry-opentelemetry-node-\d.*\.tgz$/
- name: npm
id: '@sentry/bun'
includeNames: /^sentry-bun-\d.*\.tgz$/
Expand Down Expand Up @@ -194,8 +191,6 @@ targets:
onlyIfPresent: /^sentry-svelte-\d.*\.tgz$/
'npm:@sentry/sveltekit':
onlyIfPresent: /^sentry-sveltekit-\d.*\.tgz$/
'npm:@sentry/opentelemetry-node':
onlyIfPresent: /^sentry-opentelemetry-node-\d.*\.tgz$/
'npm:@sentry/bun':
onlyIfPresent: /^sentry-bun-\d.*\.tgz$/
'npm:@sentry/vercel-edge':
Expand Down
3 changes: 1 addition & 2 deletions .github/ISSUE_TEMPLATE/bug.yml
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ body:
setup.
options:
- '@sentry/browser'
- '@sentry/astro'
- '@sentry/astro'
- '@sentry/angular'
- '@sentry/angular-ivy'
- '@sentry/bun'
Expand All @@ -40,7 +40,6 @@ body:
- '@sentry/gatsby'
- '@sentry/nextjs'
- '@sentry/node'
- '@sentry/opentelemetry-node'
- '@sentry/react'
- '@sentry/remix'
- '@sentry/serverless'
Expand Down
3 changes: 0 additions & 3 deletions .github/workflows/issue-package-label.yml
Original file line number Diff line number Diff line change
Expand Up @@ -56,9 +56,6 @@ jobs:
"@sentry.node": {
"label": "Package: Node"
},
"@sentry.opentelemetry-node": {
"label": "Package: otel-node"
},
"@sentry.react": {
"label": "Package: react"
},
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,6 @@ import * as _SentryCore from '@sentry/core';
// biome-ignore lint/nursery/noUnusedImports:
import * as _SentryNode from '@sentry/node';
// biome-ignore lint/nursery/noUnusedImports:
import * as _SentryOpentelemetry from '@sentry/opentelemetry-node';
// biome-ignore lint/nursery/noUnusedImports:
import * as _SentryReplay from '@sentry/replay';
// biome-ignore lint/nursery/noUnusedImports:
import * as _SentryTypes from '@sentry/types';
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,6 @@
"@sentry/browser": "latest || *",
"@sentry/core": "latest || *",
"@sentry/node": "latest || *",
"@sentry/opentelemetry-node": "latest || *",
"@sentry/replay": "latest || *",
"@sentry/types": "latest || *",
"@sentry/utils": "latest || *",
Expand Down
6 changes: 0 additions & 6 deletions dev-packages/e2e-tests/verdaccio-config/config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -104,12 +104,6 @@ packages:
unpublish: $all
# proxy: npmjs # Don't proxy for E2E tests!

'@sentry/opentelemetry-node':
access: $all
publish: $all
unpublish: $all
# proxy: npmjs # Don't proxy for E2E tests!

'@sentry/opentelemetry':
access: $all
publish: $all
Expand Down
3 changes: 1 addition & 2 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@
"postpublish": "lerna run --stream --concurrency 1 postpublish",
"test": "lerna run --ignore \"@sentry-internal/{browser-integration-tests,e2e-tests,integration-shims,node-integration-tests,overhead-metrics}\" test",
"test:unit": "lerna run --ignore \"@sentry-internal/{browser-integration-tests,e2e-tests,integration-shims,node-integration-tests,overhead-metrics}\" test:unit",
"test-ci-browser": "lerna run test --ignore \"@sentry/{bun,deno,node,node-experimental,opentelemetry-node,profiling-node,serverless,nextjs,remix,gatsby,sveltekit,vercel-edge}\" --ignore \"@sentry-internal/{browser-integration-tests,e2e-tests,integration-shims,node-integration-tests,overhead-metrics}\"",
"test-ci-browser": "lerna run test --ignore \"@sentry/{bun,deno,node,node-experimental,profiling-node,serverless,nextjs,remix,gatsby,sveltekit,vercel-edge}\" --ignore \"@sentry-internal/{browser-integration-tests,e2e-tests,integration-shims,node-integration-tests,overhead-metrics}\"",
"test-ci-node": "ts-node ./scripts/node-unit-tests.ts",
"test-ci-bun": "lerna run test --scope @sentry/bun",
"test:update-snapshots": "lerna run test:update-snapshots",
Expand All @@ -61,7 +61,6 @@
"packages/nextjs",
"packages/node",
"packages/node-experimental",
"packages/opentelemetry-node",
"packages/opentelemetry",
"packages/profiling-node",
"packages/react",
Expand Down
6 changes: 0 additions & 6 deletions packages/core/src/baseclient.ts
Original file line number Diff line number Diff line change
Expand Up @@ -443,9 +443,6 @@ export abstract class BaseClient<O extends ClientOptions> implements Client<O> {
/** @inheritdoc */
public on(hook: 'createDsc', callback: (dsc: DynamicSamplingContext) => void): void;

/** @inheritdoc */
public on(hook: 'otelSpanEnd', callback: (otelSpan: unknown, mutableOptions: { drop: boolean }) => void): void;

/** @inheritdoc */
public on(
hook: 'beforeSendFeedback',
Expand Down Expand Up @@ -496,9 +493,6 @@ export abstract class BaseClient<O extends ClientOptions> implements Client<O> {
/** @inheritdoc */
public emit(hook: 'createDsc', dsc: DynamicSamplingContext): void;

/** @inheritdoc */
public emit(hook: 'otelSpanEnd', otelSpan: unknown, mutableOptions: { drop: boolean }): void;

/** @inheritdoc */
public emit(hook: 'beforeSendFeedback', feedback: FeedbackEvent, options?: { includeReplay: boolean }): void;

Expand Down
9 changes: 0 additions & 9 deletions packages/opentelemetry-node/.eslintrc.js

This file was deleted.

14 changes: 0 additions & 14 deletions packages/opentelemetry-node/LICENSE

This file was deleted.

75 changes: 0 additions & 75 deletions packages/opentelemetry-node/README.md

This file was deleted.

1 change: 0 additions & 1 deletion packages/opentelemetry-node/jest.config.js

This file was deleted.

81 changes: 0 additions & 81 deletions packages/opentelemetry-node/package.json

This file was deleted.

3 changes: 0 additions & 3 deletions packages/opentelemetry-node/rollup.npm.config.mjs

This file was deleted.

9 changes: 0 additions & 9 deletions packages/opentelemetry-node/src/constants.ts

This file was deleted.

8 changes: 0 additions & 8 deletions packages/opentelemetry-node/src/debug-build.ts

This file was deleted.

5 changes: 0 additions & 5 deletions packages/opentelemetry-node/src/index.ts

This file was deleted.

Loading

0 comments on commit ef6e6e5

Please sign in to comment.