[PR #3311] [MERGED] fix: remove redundant langchain_core from install commands [closes DOC-760] #3369

Closed
opened 2026-06-05 18:22:53 -04:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/langchain-ai/docs/pull/3311
Author: @katmayb
Created: 3/25/2026
Status: Merged
Merged: 3/25/2026
Merged by: @katmayb

Base: mainHead: open-swe/2989cf58-b91f-c255-ecc3-de9f2786534e


📝 Commits (4)

  • 20fc5a9 fix: remove redundant langchain_core from install commands [closes DOC-760]
  • 8c48c0a Apply suggestions from code review
  • cb21e09 Apply suggestion from @katmayb
  • ee03d7d Merge branch 'main' into open-swe/2989cf58-b91f-c255-ecc3-de9f2786534e

📊 Changes

1 file changed (+2 additions, -2 deletions)

View changed files

📝 src/langsmith/trace-with-langchain.mdx (+2 -2)

📄 Description

Description

The trace-with-langchain page was instructing users to manually install langchain_core, which is unnecessary since it is automatically installed as a transitive dependency of langchain_openai (for just python). This caused a discrepancy with the LangSmith app's own tracing quickstart, which only installs langchain and langchain-openai.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/langchain-ai/docs/pull/3311 **Author:** [@katmayb](https://github.com/katmayb) **Created:** 3/25/2026 **Status:** ✅ Merged **Merged:** 3/25/2026 **Merged by:** [@katmayb](https://github.com/katmayb) **Base:** `main` ← **Head:** `open-swe/2989cf58-b91f-c255-ecc3-de9f2786534e` --- ### 📝 Commits (4) - [`20fc5a9`](https://github.com/langchain-ai/docs/commit/20fc5a97591adfe021bdc9b8a56cbfd801a99163) fix: remove redundant langchain_core from install commands [closes DOC-760] - [`8c48c0a`](https://github.com/langchain-ai/docs/commit/8c48c0a2814dfc67403ed4df97840b93cb403bc9) Apply suggestions from code review - [`cb21e09`](https://github.com/langchain-ai/docs/commit/cb21e09551f5d196fb28fefb320f493337a991e7) Apply suggestion from @katmayb - [`ee03d7d`](https://github.com/langchain-ai/docs/commit/ee03d7df8ee69f9ceaf409c34e966d8660121dcb) Merge branch 'main' into open-swe/2989cf58-b91f-c255-ecc3-de9f2786534e ### 📊 Changes **1 file changed** (+2 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `src/langsmith/trace-with-langchain.mdx` (+2 -2) </details> ### 📄 Description ## Description The `trace-with-langchain` page was instructing users to manually install `langchain_core`, which is unnecessary since it is automatically installed as a transitive dependency of `langchain_openai` (for just python). This caused a discrepancy with the LangSmith app's own tracing quickstart, which only installs `langchain` and `langchain-openai`. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-06-05 18:22:53 -04:00
yindo closed this issue 2026-06-05 18:22:53 -04:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: langchain-ai/docs#3369