Skip to content

fix(release): remove follow-up mention notification - #373

Merged
vince-hz merged 1 commit into
mainfrom
codex/remove-release-followup
Oct 10, 2026
Merged

vince-hz merged 1 commit into
mainfrom
codex/remove-release-followup

Conversation

@vince-hz

Copy link
Copy Markdown
Contributor

删除 release 后单独发送的 @Lava 更新 oo-cli 提醒,保留正常的版本发布通知。同步清理提醒消息构造函数、相关测试及 workflow 中的用户 ID 配置。

验证:bun run lint:fix、bun run ts-check、bun run knip 均通过;bun run test ./contrib/ci/release-workflow.test.ts 12 项通过。

@coderabbitai

coderabbitai Bot commented Oct 10, 2026 •

Copy link
Copy Markdown

Review in Change Stack →Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 10fb109d-4c11-41ee-a6d3-8464d5ccdf92

📥 Commits

Reviewing files that changed from the base of the PR and between d80e0ff and 9c92b5a.


📒 Files selected for processing (4)
  • .github/workflows/publish.yaml
  • contrib/ci/release-steps.ts
  • contrib/ci/release-workflow.test.ts
  • contrib/ci/release-workflow.ts

💤 Files with no reviewable changes (4)
  • .github/workflows/publish.yaml
  • contrib/ci/release-workflow.ts
  • contrib/ci/release-workflow.test.ts
  • contrib/ci/release-steps.ts

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.



Summary by CodeRabbit

  • Changes
    • Release notifications continue to be sent through Feishu. The separate follow-up message is no longer sent.

Walkthrough

The release workflow no longer builds or sends a separate Feishu follow-up message. The associated notification builder, escaping helper, environment variable assignment, and tests were removed. The release notification remains.

Priority: ⬇️ Low

Merge Risk: ⚪ Minimal · up to 9c92b

The separate reminder is removed while the normal release notification remains. No actionable merge-blocking risk is identified.

Pre-merge checks | Passed 4
✅ Passed checks (4 passed)
Check name Status Explanation
Title check Passed The title follows the required (): format, uses English, and accurately describes the removal of the follow-up mention notification.
Description check Passed The description directly explains the removal of the separate release reminder, the retained release notification, and the related cleanup. It also reports validation results.
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.


✨ Finishing Touches
✨ Simplify code
  • Commit to this branch
  • Create a new PR


  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

@vince-hz
vince-hz merged commit a0af90f into main Oct 10, 2026
6 checks passed
@vince-hz
vince-hz deleted the codex/remove-release-followup branch October 10, 2026 02:47
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.

1 participant