Skip to content

docs: document Node.js module lookup path#5087

Merged
swiatekm merged 1 commit into
open-telemetry:mainfrom
popsiclelmlm:docs/nodejs-winston-node-path
May 18, 2026
Merged

docs: document Node.js module lookup path#5087
swiatekm merged 1 commit into
open-telemetry:mainfrom
popsiclelmlm:docs/nodejs-winston-node-path

Conversation

@popsiclelmlm
Copy link
Copy Markdown
Contributor

Fixes #3653

Summary

  • explain when Node.js auto-instrumentation may need NODE_PATH to find application dependencies
  • add an Instrumentation.spec.nodejs.env example for setting NODE_PATH

Verification

  • git diff --check

@popsiclelmlm popsiclelmlm requested a review from a team as a code owner May 17, 2026 17:04
Copy link
Copy Markdown
Contributor

@swiatekm swiatekm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the contribution!

@swiatekm swiatekm added the Skip Changelog PRs that do not require a CHANGELOG.md entry label May 17, 2026
@swiatekm swiatekm enabled auto-merge (squash) May 17, 2026 20:41
@swiatekm swiatekm merged commit 6ff6e0d into open-telemetry:main May 18, 2026
96 of 100 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Skip Changelog PRs that do not require a CHANGELOG.md entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add documentation for instrumentation-winston lookup path

3 participants