[PR #898] [MERGED] Fix broken links and fail on new broken links #1128

Closed
opened 2026-02-17 17:21:46 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/langchain-ai/docs/pull/898
Author: @npentrel
Created: 10/14/2025
Status: Merged
Merged: 10/15/2025
Merged by: @lnhsingh

Base: mainHead: add-link-checking


📝 Commits (10+)

📊 Changes

98 files changed (+722 additions, -208 deletions)

View changed files

📝 .github/workflows/check-links.yml (+3 -8)
📝 .github/workflows/ci.yml (+2 -2)
📝 Makefile (+1 -1)
📝 pipeline/core/builder.py (+19 -2)
📝 src/langsmith/application-structure.mdx (+8 -2)
📝 src/langsmith/deploy-hybrid.mdx (+1 -1)
📝 src/langsmith/deploy-self-hosted-full-platform.mdx (+1 -1)
📝 src/langsmith/egress-metrics-metadata.mdx (+1 -1)
📝 src/langsmith/hosting.mdx (+1 -1)
📝 src/langsmith/hybrid.mdx (+1 -1)
📝 src/langsmith/integrations.mdx (+1 -1)
📝 src/langsmith/prompt-engineering.mdx (+1 -1)
📝 src/langsmith/setup-app-requirements-txt.mdx (+8 -2)
📝 src/langsmith/setup-javascript.mdx (+8 -2)
📝 src/langsmith/setup-pyproject.mdx (+8 -2)
📝 src/langsmith/use-studio.mdx (+1 -1)
📝 src/oss/concepts/context.mdx (+8 -2)
📝 src/oss/concepts/memory.mdx (+9 -3)
📝 src/oss/contributing/implement-langchain.mdx (+3 -3)
📝 src/oss/contributing/integrations-langchain.mdx (+4 -4)

...and 78 more files

📄 Description

Overview

This updates a number of broken links and adds a GitHub action that fails if there are more than 30 broken links. This is an arbitrary number based on a number of template links and a few broken links I don't know how to fix.

Type of change

Type: bug/other

Checklist

  • I have read the contributing guidelines
  • I have tested my changes locally using docs dev
  • All code examples have been tested and work correctly
  • I have used root relative paths for internal links
  • I have updated navigation in src/docs.json if needed
  • I have gotten approval from the relevant reviewers

Additional notes

If you can add the hacktoberfest-accepted label to this PR that would be appreciated! Thanks!


🔄 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/898 **Author:** [@npentrel](https://github.com/npentrel) **Created:** 10/14/2025 **Status:** ✅ Merged **Merged:** 10/15/2025 **Merged by:** [@lnhsingh](https://github.com/lnhsingh) **Base:** `main` ← **Head:** `add-link-checking` --- ### 📝 Commits (10+) - [`3cfa958`](https://github.com/langchain-ai/docs/commit/3cfa958a5e595cf577ada4cb8e26f624943a69ac) Add link checking and fix some links - [`0fd85df`](https://github.com/langchain-ai/docs/commit/0fd85df1ed5cd3ae29049d721f31fecb754107ba) Fail on more than 30 broken links - [`0d1f2c5`](https://github.com/langchain-ai/docs/commit/0d1f2c55108801a62340cdf50eb012f993d66fd6) fix docs.json - [`4a613ee`](https://github.com/langchain-ai/docs/commit/4a613eeece65f066d0773d510024df949d029046) Update the GH actions - [`86b0abc`](https://github.com/langchain-ai/docs/commit/86b0abc3208eba54f7b45c210be6ca030c605a73) Fix template errors - [`67e8f57`](https://github.com/langchain-ai/docs/commit/67e8f57333966cdad15a93ca0dda5cb653a90bc0) Fix snippets errors - [`113ea05`](https://github.com/langchain-ai/docs/commit/113ea05ad36fe9be451ba40f47a277464bcf85df) Update remaining snippets - [`5da845f`](https://github.com/langchain-ai/docs/commit/5da845fbf4aace6594320cf53bda68a0ad4ecc32) Lint - [`c390266`](https://github.com/langchain-ai/docs/commit/c3902665500c6ddaa3987eee71654d95b7548927) Fix new broken links - [`6d1fbef`](https://github.com/langchain-ai/docs/commit/6d1fbefb4b2675503b5fe5c0f2c44a2f3e2f3732) Merge branch 'main' into add-link-checking ### 📊 Changes **98 files changed** (+722 additions, -208 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/check-links.yml` (+3 -8) 📝 `.github/workflows/ci.yml` (+2 -2) 📝 `Makefile` (+1 -1) 📝 `pipeline/core/builder.py` (+19 -2) 📝 `src/langsmith/application-structure.mdx` (+8 -2) 📝 `src/langsmith/deploy-hybrid.mdx` (+1 -1) 📝 `src/langsmith/deploy-self-hosted-full-platform.mdx` (+1 -1) 📝 `src/langsmith/egress-metrics-metadata.mdx` (+1 -1) 📝 `src/langsmith/hosting.mdx` (+1 -1) 📝 `src/langsmith/hybrid.mdx` (+1 -1) 📝 `src/langsmith/integrations.mdx` (+1 -1) 📝 `src/langsmith/prompt-engineering.mdx` (+1 -1) 📝 `src/langsmith/setup-app-requirements-txt.mdx` (+8 -2) 📝 `src/langsmith/setup-javascript.mdx` (+8 -2) 📝 `src/langsmith/setup-pyproject.mdx` (+8 -2) 📝 `src/langsmith/use-studio.mdx` (+1 -1) 📝 `src/oss/concepts/context.mdx` (+8 -2) 📝 `src/oss/concepts/memory.mdx` (+9 -3) 📝 `src/oss/contributing/implement-langchain.mdx` (+3 -3) 📝 `src/oss/contributing/integrations-langchain.mdx` (+4 -4) _...and 78 more files_ </details> ### 📄 Description ## Overview This updates a number of broken links and adds a GitHub action that fails if there are more than 30 broken links. This is an arbitrary number based on a number of template links and a few broken links I don't know how to fix. ## Type of change **Type:** bug/other ## Checklist - [x] I have read the [contributing guidelines](README.md) - [x] I have tested my changes locally using `docs dev` - [x] All code examples have been tested and work correctly - [x] I have used **root relative** paths for internal links - [x] I have updated navigation in `src/docs.json` if needed - [x] I have gotten approval from the relevant reviewers ## Additional notes >If you can add the `hacktoberfest-accepted` label to this PR that would be appreciated! Thanks! --- <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-02-17 17:21:46 -05:00
yindo closed this issue 2026-02-17 17:21:46 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: langchain-ai/docs#1128