From dd7438d1ce507c601483bbaefa97b50d1dc7c55f Mon Sep 17 00:00:00 2001 From: Marc Pichler Date: Wed, 16 Aug 2023 14:13:57 +0200 Subject: [PATCH 1/5] chore: add notice that '@opentelemetry/exporter-jaeger' has to be installed manually in the next version --- experimental/CHANGELOG.md | 5 +++++ experimental/packages/opentelemetry-sdk-node/package.json | 3 ++- 2 files changed, 7 insertions(+), 1 deletion(-) diff --git a/experimental/CHANGELOG.md b/experimental/CHANGELOG.md index 64b4debd38..c8848d144b 100644 --- a/experimental/CHANGELOG.md +++ b/experimental/CHANGELOG.md @@ -8,6 +8,11 @@ All notable changes to experimental packages in this project will be documented ### :boom: Breaking Change +* chore(sdk-node): add notice that '@opentelemetry/exporter-jaeger' has to be installed manually in the next version [#????](https://github.com/open-telemetry/opentelemetry-js/pull/????) @pichlermarc + * Starting with 0.43.0 '@opentelemetry/exporter-jaeger' will no longer be a dependency of this package. To continue using '@opentelemetry/exporter-jaeger', please install it manually. + * NOTE: `@opentelemetry/exporter-jaeger` is deprecated, consider switching to one of the alternatives described in https://www.npmjs.com/package/@opentelemetry/exporter-jaeger + + ### :rocket: (Enhancement) ### :bug: (Bug Fix) diff --git a/experimental/packages/opentelemetry-sdk-node/package.json b/experimental/packages/opentelemetry-sdk-node/package.json index 857c8c79d9..e1e14fb3b6 100644 --- a/experimental/packages/opentelemetry-sdk-node/package.json +++ b/experimental/packages/opentelemetry-sdk-node/package.json @@ -17,7 +17,8 @@ "watch": "tsc --build --watch", "precompile": "cross-var lerna run version --scope $npm_package_name --include-dependencies", "prewatch": "npm run precompile", - "peer-api-check": "node ../../../scripts/peer-api-check.js" + "peer-api-check": "node ../../../scripts/peer-api-check.js", + "postinstall": "node -e \"console.log(\\\"\\x1b[95m%s\\x1b[0m\\\", \\\"@opentelemetry/sdk-node - warning: Starting with 0.43.0 '@opentelemetry/exporter-jaeger' will no longer be a dependency of this package. To continue using '@opentelemetry/exporter-jaeger', please install it manually.\\n(NOTE: '@opentelemetry/exporter-jaeger' is deprecated, consider switching to one of the alternatives described in https://www.npmjs.com/package/@opentelemetry/exporter-jaeger)\\\");\"" }, "keywords": [ "opentelemetry", From 23b10fb9303037de062fd47081b884623372da1e Mon Sep 17 00:00:00 2001 From: Marc Pichler Date: Wed, 16 Aug 2023 14:22:40 +0200 Subject: [PATCH 2/5] fix(changlog): add pr number --- experimental/CHANGELOG.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/experimental/CHANGELOG.md b/experimental/CHANGELOG.md index c8848d144b..d60f2cf82b 100644 --- a/experimental/CHANGELOG.md +++ b/experimental/CHANGELOG.md @@ -8,7 +8,7 @@ All notable changes to experimental packages in this project will be documented ### :boom: Breaking Change -* chore(sdk-node): add notice that '@opentelemetry/exporter-jaeger' has to be installed manually in the next version [#????](https://github.com/open-telemetry/opentelemetry-js/pull/????) @pichlermarc +* chore(sdk-node): add notice that '@opentelemetry/exporter-jaeger' has to be installed manually in the next version [4068](https://github.com/open-telemetry/opentelemetry-js/pull/4068) @pichlermarc * Starting with 0.43.0 '@opentelemetry/exporter-jaeger' will no longer be a dependency of this package. To continue using '@opentelemetry/exporter-jaeger', please install it manually. * NOTE: `@opentelemetry/exporter-jaeger` is deprecated, consider switching to one of the alternatives described in https://www.npmjs.com/package/@opentelemetry/exporter-jaeger From 0653dd0e5f4a39d98429d797557395986bea6f21 Mon Sep 17 00:00:00 2001 From: Marc Pichler Date: Wed, 16 Aug 2023 14:24:09 +0200 Subject: [PATCH 3/5] fix(changlog): fix indentation --- experimental/CHANGELOG.md | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/experimental/CHANGELOG.md b/experimental/CHANGELOG.md index d60f2cf82b..8c2bfdf284 100644 --- a/experimental/CHANGELOG.md +++ b/experimental/CHANGELOG.md @@ -9,9 +9,8 @@ All notable changes to experimental packages in this project will be documented ### :boom: Breaking Change * chore(sdk-node): add notice that '@opentelemetry/exporter-jaeger' has to be installed manually in the next version [4068](https://github.com/open-telemetry/opentelemetry-js/pull/4068) @pichlermarc - * Starting with 0.43.0 '@opentelemetry/exporter-jaeger' will no longer be a dependency of this package. To continue using '@opentelemetry/exporter-jaeger', please install it manually. - * NOTE: `@opentelemetry/exporter-jaeger` is deprecated, consider switching to one of the alternatives described in https://www.npmjs.com/package/@opentelemetry/exporter-jaeger - + * Starting with 0.43.0 '@opentelemetry/exporter-jaeger' will no longer be a dependency of this package. To continue using '@opentelemetry/exporter-jaeger', please install it manually. + * NOTE: `@opentelemetry/exporter-jaeger` is deprecated, consider switching to one of the alternatives described in https://www.npmjs.com/package/@opentelemetry/exporter-jaeger ### :rocket: (Enhancement) From c4a9a492a990545f2735a1f4936dc15c389bc813 Mon Sep 17 00:00:00 2001 From: Marc Pichler Date: Wed, 16 Aug 2023 14:25:49 +0200 Subject: [PATCH 4/5] fix(changlog): fix link --- experimental/CHANGELOG.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/experimental/CHANGELOG.md b/experimental/CHANGELOG.md index 8c2bfdf284..905f744d87 100644 --- a/experimental/CHANGELOG.md +++ b/experimental/CHANGELOG.md @@ -10,7 +10,7 @@ All notable changes to experimental packages in this project will be documented * chore(sdk-node): add notice that '@opentelemetry/exporter-jaeger' has to be installed manually in the next version [4068](https://github.com/open-telemetry/opentelemetry-js/pull/4068) @pichlermarc * Starting with 0.43.0 '@opentelemetry/exporter-jaeger' will no longer be a dependency of this package. To continue using '@opentelemetry/exporter-jaeger', please install it manually. - * NOTE: `@opentelemetry/exporter-jaeger` is deprecated, consider switching to one of the alternatives described in https://www.npmjs.com/package/@opentelemetry/exporter-jaeger + * NOTE: `@opentelemetry/exporter-jaeger` is deprecated, consider switching to one of the alternatives described [here](https://www.npmjs.com/package/@opentelemetry/exporter-jaeger) ### :rocket: (Enhancement) From 6cdf0f87c1c4ac15fe8da942e781f056cae5556b Mon Sep 17 00:00:00 2001 From: Marc Pichler Date: Thu, 17 Aug 2023 13:13:04 +0200 Subject: [PATCH 5/5] Update experimental/packages/opentelemetry-sdk-node/package.json Co-authored-by: Daniel Dyla --- experimental/packages/opentelemetry-sdk-node/package.json | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/experimental/packages/opentelemetry-sdk-node/package.json b/experimental/packages/opentelemetry-sdk-node/package.json index e1e14fb3b6..ef755d69bc 100644 --- a/experimental/packages/opentelemetry-sdk-node/package.json +++ b/experimental/packages/opentelemetry-sdk-node/package.json @@ -18,7 +18,7 @@ "precompile": "cross-var lerna run version --scope $npm_package_name --include-dependencies", "prewatch": "npm run precompile", "peer-api-check": "node ../../../scripts/peer-api-check.js", - "postinstall": "node -e \"console.log(\\\"\\x1b[95m%s\\x1b[0m\\\", \\\"@opentelemetry/sdk-node - warning: Starting with 0.43.0 '@opentelemetry/exporter-jaeger' will no longer be a dependency of this package. To continue using '@opentelemetry/exporter-jaeger', please install it manually.\\n(NOTE: '@opentelemetry/exporter-jaeger' is deprecated, consider switching to one of the alternatives described in https://www.npmjs.com/package/@opentelemetry/exporter-jaeger)\\\");\"" + "postinstall": "node -e \"console.log(\\\"\\x1b[95m%s\\x1b[0m\\\", \\\"@opentelemetry/sdk-node - warning: Starting with 0.43.0 '@opentelemetry/exporter-jaeger' will no longer be automatically installed by this package. To continue using '@opentelemetry/exporter-jaeger', please install it manually.\\n(NOTE: '@opentelemetry/exporter-jaeger' is deprecated, consider switching to one of the alternatives described in https://www.npmjs.com/package/@opentelemetry/exporter-jaeger)\\\");\"" }, "keywords": [ "opentelemetry",