mattuy/text-mosaic
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(ci): 修复通知优先级被预展开 #8

Merged
mattuy merged 1 commits from fix/woodpecker-success-priority into main 2026-08-01 01:02:54 +00:00
Conversation 0 Commits 1 Files Changed 1
+1 -1
mattuy commented 2026-08-01 00:57:26 +00:00
Owner
Copy Link
Copy Source

问题：Woodpecker 会在 step 执行前展开 ${NOTIFY_PRIORITY:-}，导致容器内拿到的 priority 为空，成功通知最终走默认 normal。

修复：改用 $NOTIFY_PRIORITY，把变量读取留到容器 shell 运行时；开始通知继续为 silent，成功通知为 low，失败通知仍不传 priority。

验证：Woodpecker 配置 lint 通过。

问题：Woodpecker 会在 step 执行前展开 `${NOTIFY_PRIORITY:-}`，导致容器内拿到的 priority 为空，成功通知最终走默认 normal。 修复：改用 `$NOTIFY_PRIORITY`，把变量读取留到容器 shell 运行时；开始通知继续为 silent，成功通知为 low，失败通知仍不传 priority。 验证：Woodpecker 配置 lint 通过。
mattuy added 1 commit 2026-08-01 00:57:28 +00:00
fix(ci): preserve notification priority at runtime 88bb2f1542
mattuy merged commit 021d96c201 into main 2026-08-01 01:02:54 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
mattuy (Haven Mattuy)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: mattuy/text-mosaic#8
Delete Branch "fix/woodpecker-success-priority"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?