[PR #1630] [MERGED] refactor(infra): factor out PR commenting to seperate workflow part #1714

Closed
opened 2026-02-17 17:22:39 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/langchain-ai/docs/pull/1630
Author: @mdrxy
Created: 11/25/2025
Status: Merged
Merged: 11/25/2025
Merged by: @mdrxy

Base: mainHead: mdrxy/fix-comment-workflow


📝 Commits (5)

  • 1ad25bf refactor(infra): factor out PR commenting to seperate workflow part
  • 0620151 Merge branch 'main' into mdrxy/fix-comment-workflow
  • 9699c12 em-dashify
  • 492e1c9 .
  • e411918 use env var to prevent untrusted input

📊 Changes

9 files changed (+166 additions, -109 deletions)

View changed files

.github/workflows/check-pr-imports-comment.yml (+138 -0)
📝 .github/workflows/check-pr-imports.yml (+12 -95)
📝 src/oss/javascript/releases/changelog.mdx (+3 -2)
📝 src/oss/javascript/releases/langchain-v1.mdx (+2 -2)
📝 src/oss/javascript/releases/langgraph-v1.mdx (+2 -2)
📝 src/oss/python/releases/changelog.mdx (+3 -2)
📝 src/oss/python/releases/langchain-v1.mdx (+2 -2)
📝 src/oss/python/releases/langgraph-v1.mdx (+2 -2)
📝 src/oss/releases.mdx (+2 -2)

📄 Description

Necessary for forks so as to not give too much permissions to primary workflow

Primary workflow doesn't require write permissions

The fork's code never runs with write permissions


🔄 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/1630 **Author:** [@mdrxy](https://github.com/mdrxy) **Created:** 11/25/2025 **Status:** ✅ Merged **Merged:** 11/25/2025 **Merged by:** [@mdrxy](https://github.com/mdrxy) **Base:** `main` ← **Head:** `mdrxy/fix-comment-workflow` --- ### 📝 Commits (5) - [`1ad25bf`](https://github.com/langchain-ai/docs/commit/1ad25bfc68d625a03991f01df023af6b4410713b) refactor(infra): factor out PR commenting to seperate workflow part - [`0620151`](https://github.com/langchain-ai/docs/commit/0620151cdf7f4e052cf600a4ebfa0f773fd62814) Merge branch 'main' into mdrxy/fix-comment-workflow - [`9699c12`](https://github.com/langchain-ai/docs/commit/9699c12917c0f077d5480da36cf4237575a9a5a1) em-dashify - [`492e1c9`](https://github.com/langchain-ai/docs/commit/492e1c9d8d2d0a2ce76b1e6b8332f4e1de9fd12e) . - [`e411918`](https://github.com/langchain-ai/docs/commit/e4119183044e539a2a0764ba65bd1d106a2635bd) use env var to prevent untrusted input ### 📊 Changes **9 files changed** (+166 additions, -109 deletions) <details> <summary>View changed files</summary> ➕ `.github/workflows/check-pr-imports-comment.yml` (+138 -0) 📝 `.github/workflows/check-pr-imports.yml` (+12 -95) 📝 `src/oss/javascript/releases/changelog.mdx` (+3 -2) 📝 `src/oss/javascript/releases/langchain-v1.mdx` (+2 -2) 📝 `src/oss/javascript/releases/langgraph-v1.mdx` (+2 -2) 📝 `src/oss/python/releases/changelog.mdx` (+3 -2) 📝 `src/oss/python/releases/langchain-v1.mdx` (+2 -2) 📝 `src/oss/python/releases/langgraph-v1.mdx` (+2 -2) 📝 `src/oss/releases.mdx` (+2 -2) </details> ### 📄 Description Necessary for forks so as to not give too much permissions to primary workflow Primary workflow doesn't require write permissions The fork's code never runs with write permissions --- <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:22:39 -05:00
yindo closed this issue 2026-02-17 17:22:39 -05:00
yindo changed title from [PR #1630] refactor(infra): factor out PR commenting to seperate workflow part to [PR #1630] [MERGED] refactor(infra): factor out PR commenting to seperate workflow part 2026-06-05 18:14:32 -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#1714