Skip to content

Conversation

@ThirteenLLB
Copy link
Contributor

@ThirteenLLB ThirteenLLB commented Jan 30, 2026

Summary

  • echo 替换为 heredoc,修复 GitHub Actions 多行输入换行符丢失的问题
  • 修复后 Centralized Release 工作流可正确解析多仓库配置

Test plan

自测环境:

  • GitHub Actions workflow_dispatch

验证步骤:

  1. 触发 Centralized Release workflow
  2. 输入多个仓库配置(每行一个)
  3. 勾选 dry_run 验证 parse 步骤正确解析所有仓库

结果:
✅ 多行输入被正确保留
✅ parse-repos.py 正确解析所有仓库

🤖 Generated with Claude Code

Summary by CodeRabbit

发布说明

  • Bug Fixes
    • 修复了集中发布工作流中多行仓库输入被折叠为单行的问题,确保输入格式和换行符得到正确保留。

✏️ Tip: You can customize this high-level summary in your review settings.

Use heredoc instead of echo to pass the repositories input to
parse-repos.py, preventing GitHub Actions from collapsing newlines
into spaces.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 30, 2026

Walkthrough

修改了中央发布工作流中存储库输入的传递方式,从管道改为使用heredoc语法,并更新了CHANGELOG以记录此修复。功能输出保持不变。

Changes

Cohort / File(s) Summary
工作流输入传递方式
.github/workflows/centralized-release.yml
将存储库输入从管道传递改为heredoc方式,保留格式和换行符,然后写入repos.json。
变更日志
CHANGELOG.md
在Unreleased下的Fixed部分添加条目,说明centralized-release.yml中多行存储库输入被折叠为单行的问题已修复。

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

Suggested reviewers

  • yuanyuanxin

Poem

🐰 heredoc助我舞,
换个方式传数据,
格式完整不折损,
工作流更优雅呀!

🚥 Pre-merge checks | ✅ 6
✅ Passed checks (6 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PR标题遵循Conventional Commits格式,使用"fix(ci)"前缀,主题清晰具体地描述了修复多行输入保留的问题,与changeset的主要变更完全相关。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Changelog ✅ Passed 该PR修复了centralized-release.yml中多行仓库输入被折叠的bug,是用户可见的变化。CHANGELOG.md已在Unreleased的Fixed小节中记录了该修复。
Pr Description ✅ Passed PR描述满足两个要求。PR objectives中包含了"Test plan"部分,包含具体的测试环境、步骤和验证结果;源分支名称中包含了工作项ID "f-31",符合"f-"的格式要求。
Branch Name ✅ Passed 源分支名称 fix/f-31/centralized-release-multiline-input 正确遵循了错误修复命名模式 fix/f-<number>/<description>

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/f-31/centralized-release-multiline-input

Comment @coderabbitai help to get the list of available commands and usage tips.

@ThirteenLLB ThirteenLLB requested a review from chenjunnn January 30, 2026 08:15
@ThirteenLLB ThirteenLLB merged commit db82f6c into main Jan 30, 2026
1 check passed
@ThirteenLLB ThirteenLLB deleted the fix/f-31/centralized-release-multiline-input branch January 30, 2026 08:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants