-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
feat(deps): Bump OpenTelemetry dependencies #18878
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
size-limit report 📦
|
node-overhead report 🧳Note: This is a synthetic benchmark with a minimal express app and does not necessarily reflect the real-world performance impact in an application.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
d826f1d to
ccccb05
Compare
- @opentelemetry/context-async-hooks - @opentelemetry/core - @opentelemetry/resources - @opentelemetry/sdk-trace-base - @opentelemetry/sdk-trace-node - @opentelemetry/instrumentation: 0.208.0 → 0.210.0 - @opentelemetry/instrumentation-amqplib: 0.55.0 → 0.57.0 - @opentelemetry/instrumentation-connect: 0.52.0 → 0.53.0 - @opentelemetry/instrumentation-dataloader: 0.26.0 → 0.27.0 - @opentelemetry/instrumentation-express: 0.57.0 → 0.58.0 - @opentelemetry/instrumentation-fs: 0.28.0 → 0.29.0 - @opentelemetry/instrumentation-generic-pool: 0.52.0 → 0.53.0 - @opentelemetry/instrumentation-graphql: 0.56.0 → 0.57.0 - @opentelemetry/instrumentation-hapi: 0.55.0 → 0.56.0 - @opentelemetry/instrumentation-http: 0.208.0 → 0.210.0 - @opentelemetry/instrumentation-ioredis: 0.56.0 → 0.58.0 - @opentelemetry/instrumentation-kafkajs: 0.18.0 → 0.19.0 - @opentelemetry/instrumentation-knex: 0.53.0 → 0.54.0 - @opentelemetry/instrumentation-koa: 0.57.0 → 0.58.0 - @opentelemetry/instrumentation-lru-memoizer: 0.53.0 → 0.54.0 - @opentelemetry/instrumentation-mongodb: 0.61.0 → 0.63.0 - @opentelemetry/instrumentation-mongoose: 0.55.0 → 0.56.0 - @opentelemetry/instrumentation-mysql: 0.54.0 → 0.56.0 - @opentelemetry/instrumentation-mysql2: 0.55.0 → 0.56.0 - @opentelemetry/instrumentation-nestjs-core: 0.55.0 → 0.56.0 - @opentelemetry/instrumentation-pg: 0.61.0 → 0.62.0 - @opentelemetry/instrumentation-redis: 0.57.0 → 0.58.0 - @opentelemetry/instrumentation-tedious: 0.27.0 → 0.29.0 - @opentelemetry/instrumentation-undici: 0.19.0 → 0.20.0 - @opentelemetry/instrumentation-aws-sdk: 0.64.0 → 0.65.0 - @opentelemetry/sdk-node: 0.208.0 → 0.210.0 - @opentelemetry/exporter-trace-otlp-http: 0.208.0 → 0.210.0
ccccb05 to
cad2fac
Compare
| preventAssignment: true, | ||
| // Use negative lookahead/lookbehind instead of word boundaries so `process.argv0` is also replaced in | ||
| // `process.argv0.length` (where `.` follows). Default `\b` delimiters don't match before `.`. | ||
| delimiters: ['(?<![\\w$])', '(?![\\w$])'], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
l: Why has this been added?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was added because with 2.4.0, the way they are using process.argv0 has changed and the regex had to be adjusted to work for it.
|
|
||
| ### Other Changes | ||
|
|
||
| - feat(deps): Bump OpenTelemetry dependencies |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Want to keep this all in the changelog?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry didn't see this comment. I aligned this with how dependabot PRs end up in the changelog, one entry per dep with clear statement of what was bumped from what.
We could also just leave it in the PR description but I preferred to be explicit. Happy to change in the future tho.
Closes #18874