Skip to content

Conversation

@petyosi
Copy link
Contributor

@petyosi petyosi commented Nov 19, 2025

No description provided.

@petyosi petyosi requested a review from Copilot November 19, 2025 10:27
Copilot finished reviewing on behalf of petyosi November 19, 2025 10:28
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the documentation to reflect new JavaScript package names for Logfire. The old logfire package is being renamed to @pydantic/logfire-node for Node.js usage.

Key changes:

  • Updated package installation and import statements in Node.js documentation
  • Updated package name references in the JavaScript integrations overview
  • Minor code formatting improvements for consistency

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
docs/integrations/javascript/node.md Updated package name from logfire to @pydantic/logfire-node in installation command and import statement
docs/integrations/javascript/index.md Updated package name references to reflect the new naming convention

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Nov 19, 2025

Deploying logfire-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: aa935c2
Status: ✅  Deploy successful!
Preview URL: https://7ab47de7.logfire-docs.pages.dev
Branch Preview URL: https://petyo-update-js-package-name.logfire-docs.pages.dev

View logs

@petyosi petyosi force-pushed the petyo/update-js-package-names branch from d4b8db4 to aa935c2 Compare November 19, 2025 10:33
@alexmojaki alexmojaki merged commit 370f763 into main Nov 19, 2025
14 checks passed
@alexmojaki alexmojaki deleted the petyo/update-js-package-names branch November 19, 2025 10:45
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.

3 participants