Skip to content

feat(ai-monitoring): Langchain integration docs - #9799

Merged
colin-sentry merged 4 commits into
masterfrom
langchain
Apr 25, 2024
Merged

feat(ai-monitoring): Langchain integration docs#9799
colin-sentry merged 4 commits into
masterfrom
langchain

Conversation

@colin-sentry

@colin-sentrycolin-sentry commented Apr 24, 2024•
edited
Loading

Copy link
Copy Markdown
Contributor

Pre-merge checklist

If you work at Sentry, you're able to merge your own PR without review, but please don't unless there's a good reason.

  • Checked Vercel preview for correctness, including links
  • PR was reviewed and approved by any necessary SMEs
  • PR was reviewed and approved by a member of the Sentry docs team

@vercel

vercelBot commented Apr 24, 2024•
edited
Loading

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

NameStatusPreviewCommentsUpdated (UTC)
sentry-docs✅ Ready (Inspect)Visit Preview💬 Add feedbackApr 25, 2024 5:11pm

@colin-sentrycolin-sentry mentioned this pull request Apr 24, 2024
Merged
@codecov

codecovBot commented Apr 24, 2024•
edited
Loading

Copy link
Copy Markdown

Bundle Report

Changes will increase total bundle size by 256 bytes ⬆️

Bundle nameSizeChange
sentry-docs-server7.12MB235 bytes ⬆️
sentry-docs-client5.76MB6 bytes ⬇️
sentry-docs-edge-server427.92kB27 bytes ⬆️

@colin-sentry
colin-sentry requested a review from a teamApril 24, 2024 15:26
@colin-sentrycolin-sentry changed the title Langchain integration docsfeat(ai-monitoring): Langchain integration docsApr 24, 2024
@vivianyentran
vivianyentran self-requested a review April 24, 2024 19:08
vivianyentran
vivianyentran suggested changes Apr 24, 2024
View reviewed changes

@vivianyentranvivianyentran left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for adding this! Added some suggestions

Comment threaddocs/platforms/python/integrations/langchain/index.mdx Outdated
Comment threaddocs/platforms/python/integrations/langchain/index.mdx Outdated
Comment threaddocs/platforms/python/integrations/langchain/index.mdx
Comment threaddocs/platforms/python/integrations/langchain/index.mdx Outdated
vivianyentran
vivianyentran approved these changes Apr 25, 2024
View reviewed changes

@vivianyentranvivianyentran left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Left another suggestion but overall, looks good!

Comment threaddocs/platforms/python/integrations/langchain/index.mdx Outdated
@colin-sentry@vivianyentran
Update docs/platforms/python/integrations/langchain/index.mdx
8e8a68c
Co-authored-by: vivianyentran <20403606+vivianyentran@users.noreply.github.com>
@colin-sentry
colin-sentry enabled auto-merge (squash) April 25, 2024 16:56
@colin-sentry
colin-sentry merged commit f4da261 into masterApr 25, 2024
@colin-sentry
colin-sentry deleted the langchain branch April 25, 2024 17:11
@github-actionsgithub-actionsBot locked and limited conversation to collaborators May 11, 2024
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer
@vivianyentranvivianyentranvivianyentran approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@colin-sentry@vivianyentran