Skip to content

Conversation

@Calm-Rock
Copy link
Member

@Calm-Rock Calm-Rock commented Aug 13, 2025

Fixes #1546

@vercel
Copy link

vercel bot commented Aug 13, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
signoz-web Ready Ready Preview Comment Sep 1, 2025 6:42am

@Calm-Rock Calm-Rock changed the title docs: Add Node.js Pino auto-instrumentation guide docs: Update Node.js Pino guide for auto-instrumentation with traces and logs correlation Aug 13, 2025
- Reduce intro section to 2-3 lines 
- Change "guide" to "document"
- Replace dropdowns with tab components in Step 2
- Add tab components to Step 3 for better organization
- Load environment variables directly in otel.js/otel.ts files
- Add validation and error handling for required variables
- More explicit control over exporter configuration
- Better debugging and troubleshooting capabilities
- Keep otel.js/otel.ts files simple and focused
- Environment variables handled externally as originally intended
- Maintain tab components and formatting improvements
- Cleaner separation of concerns
- Combine Step 3 and Configuration section into single step
- Remove service mapping reference from verification 
- Remove debug mode section for cleaner documentation
- Update intro to focus on logging rather than tracing
- Use single endpoint for both Cloud and self-hosted setups
- Add consistent placeholder format with explanatory bullet points
- Include complete examples in Start Your Application section
Copy link
Member

@nityanandagohain nityanandagohain left a comment

Choose a reason for hiding this comment

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

share the sample app.js /app.ts which can be run while following this doc as that will help.

and remove the Prerequisites part ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Use autoinstrumentation for pino and winston docs

2 participants