sanguo/sanguo_moziplus_v2
8
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

feat(toolchain): add PR synchronize and review comment notifications #35

Merged
admin merged 3 commits from feat/pr-lifecycle-events into main 2026-06-11 06:10:45 +00:00
Conversation 1 Commits 3 Files Changed 7
+409 -8

3 Commits

Author SHA1 Message Date
cfdaily a4bb752d71 feat(toolchain): add PR synchronize and review comment notifications
CI / lint (pull_request) Successful in 6s
Details
CI / test (pull_request) Successful in 8s
Details
CI / notify-on-failure (pull_request) Successful in 0s
Details 
- pull_request.synchronize: notify reviewer to re-review after push
- pull_request_review COMMENTED: notify PR author of review comments
- New templates: review_updated.md, review_comment.md
- Idempotency: add review ID to content dedup key
- Design doc: docs/design/23-toolchain-pr-lifecycle.md
 2026-06-11 14:00:44 +08:00
cfdaily d6612de6de fix(cd): move success notification to independent job
CI / lint (pull_request) Successful in 6s
Details
CI / test (pull_request) Successful in 9s
Details
CI / notify-on-failure (pull_request) Successful in 0s
Details 
- needs.deploy.result is not available inside steps, only in job-level context
- Split into notify-deploy-success job (symmetric with notify-deploy-failure)
- Default NOTIFY_TO to jiangwei-infra for direct push scenario
 2026-06-11 13:25:48 +08:00
cfdaily f33190dc1e feat(cd): add deploy success notification
CI / lint (pull_request) Successful in 7s
Details
CI / test (pull_request) Successful in 8s
Details
CI / notify-on-failure (pull_request) Successful in 0s
Details 
- Query merged PR author via Gitea API
- Send Mail notification to PR author + pangtong
- Non-blocking: mail failure does not affect deploy
- Uses --max-time 5 on all curl calls
 2026-06-11 13:22:12 +08:00