-
Notifications
You must be signed in to change notification settings - Fork 701
10/27 work #1146
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
10/27 work #1146
Conversation
There was a problem hiding this 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 consolidates various documentation updates and corrections across the codebase, focusing on URL updates, formatting improvements, and content reorganization. The changes support documentation consistency and improve navigation by fixing broken links and standardizing formatting patterns.
Key changes:
- Updated reference URLs for LangGraph JavaScript documentation
- Added clarifying warnings to distinguish between Groq (hardware/software company) and Grok (xAI models)
- Reorganized error documentation paths and updated redirects
- Standardized formatting in provider tables and chat model listings
Reviewed Changes
Copilot reviewed 44 out of 51 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| src/snippets/oss/groq-grok.mdx | New warning snippet to clarify Groq vs. Grok distinction |
| src/snippets/oss/grok-groq.mdx | New warning snippet for Grok models referencing xAI |
| src/oss/reference/overview.mdx | Updated LangGraph JavaScript API reference URL |
| src/oss/reference/langgraph-javascript.mdx | Updated LangGraph SDK URL in frontmatter |
| src/oss/python/releases/langchain-v1.mdx | Removed extra blank lines |
| src/oss/python/migrate/langchain-v1.mdx | Removed extra blank lines throughout |
| src/oss/python/integrations/providers/xai.mdx | Added Grok/Groq warning and reformatted intro |
| src/oss/python/integrations/providers/overview.mdx | Removed color parameter from badge URLs and reordered entries |
| src/oss/python/integrations/providers/groq.mdx | Added Groq/Grok warning snippet |
| src/oss/python/integrations/chat/xai.mdx | Added warning, updated API reference links |
| src/oss/python/integrations/chat/openai.mdx | Added Chat Completions API compatibility note |
| src/oss/python/integrations/chat/index.mdx | Removed TODO comment, restructured table, added OpenRouter example |
| src/oss/python/integrations/chat/groq.mdx | Added Groq/Grok warning snippet |
| src/oss/python/integrations/chat/google_generative_ai.mdx | Updated model name from preview to production version |
| src/oss/langgraph/use-graph-api.mdx | Fixed backticks on inline code references |
| src/oss/langgraph/common-errors.mdx | Updated error doc paths and added backticks to error codes |
| src/oss/langchain/quickstart.mdx | Removed extra blank line |
| src/oss/langchain/models.mdx | Updated links from provider overview to chat model pages |
| src/oss/langchain/mcp.mdx | Changed list formatting to use bold markdown |
| src/oss/javascript/integrations/providers/overview.mdx | Added backticks to package names |
| src/oss/javascript/integrations/chat/openai.mdx | Added compatibility notes for Chat Completions API and Azure |
| src/oss/javascript/integrations/chat/index.mdx | Restructured examples, added Chat Completions API section |
| src/langsmith/langgraph-js-ts-sdk.mdx | Updated reference URL |
| src/docs.json | Updated error doc paths and changed tab names from "Contributing" to "Contribute" |
| reference/python/*.toml | Uncommented langchain-milvus dependency |
| reference/python/mkdocs.yml | Changed validation from warn to ignore for absolute links |
| reference/python/docs/langsmith/* | Added badges, quick links, and frontmatter titles |
| reference/python/docs/integrations/langchain_milvus.md | Removed WIP note and added content reference |
| pipeline/tools/partner_pkg_table.py | Removed color parameter from badge URL template |
| pipeline/preprocessors/link_map.py | Updated Runtime interface path |
| pipeline/core/builder.py | Added detailed docstring for snippet link conversion |
| README.md, CLAUDE.md, AGENTS.md, .github/copilot-instructions.md | Added snippets section explaining link preprocessing |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 44 out of 51 changed files in this pull request and generated 1 comment.
|
Preview ID generated: preview-mdrxy1-1761596715-7e015b5 |
|
Preview ID generated: preview-mdrxy1-1761596786-7a822b2 |
langchain_milvusto refs