build icon indicating copy to clipboard operation
build copied to clipboard

chore(deps): update dependency @honeycombio/opentelemetry-node to v0.5.0

Open dgp1130 opened this issue 1 year ago โ€ข 2 comments

Summary

Fixes netlify/cli#5928

pnpm users with hoist=false are currently broken because @honeycombio/opentelemetry-node has an undeclared dependency on @opentelemtry/core (netlify/cli#5928). This was fixed in https://github.com/honeycombio/honeycomb-opentelemetry-node/pull/207 and released in v0.5.0. While the package is still in prerelease, the release notes do not declare any breaking changes, so this should be a safe upgrade.

I generated this with (cd packages/build/ && npm install @honeycombio/opentelemetry-node@^0.5.0). Unfortunately this does appear to bump some other packages, however I don't see anything concerning there.


For us to review and ship your PR efficiently, please perform the following steps:

  • [X] Open a bug/issue before writing your code ๐Ÿง‘โ€๐Ÿ’ป. This ensures we can discuss the changes and get feedback from everyone that should be involved. If you`re fixing a typo or something that`s on fire ๐Ÿ”ฅ (e.g. incident related), you can skip this step.
  • [X] Read the contribution guidelines ๐Ÿ“–. This ensures your code follows our style guide and passes our tests.
  • [X] Update or add tests (if any source code was changed or added) ๐Ÿงช
  • [X] Update or add documentation (if features were changed or added) ๐Ÿ“
  • [X] Make sure the status checks below are successful โœ…

A picture of a cute animal (not mandatory, but encouraged)

One of my cats: Bon Bon. PXL_20230624_050413433

dgp1130 avatar Aug 19 '23 06:08 dgp1130

The cat looks good! Please merge ๐Ÿ™

Leksat avatar Sep 06 '23 10:09 Leksat

Hi, thanks so much for opening this PR! In the meantime, Renovate opened + merged a PR with this update: https://github.com/netlify/build/pull/5279

I'll deploy that to the CLI + cut a release. Will ping back when that's done.

Skn0tt avatar Sep 06 '23 14:09 Skn0tt

https://github.com/netlify/build/pull/5279 was merged

mrstork avatar May 03 '24 11:05 mrstork