logo
0
1
WeChat Login

feat: 添加 CNB_NPC_TRIGGER_CONTENT 建议检查项#4

Merged
created 1 weeks ago
main
auto/feat-trigger-content-check-58df63e0
Edit
OverviewCommits
1
Files changed
2
Attachments

变更说明

参考 npc/mandate-engine#67,新增一个 lint 检查项:

建议使用 CNB_NPC_TRIGGER_CONTENT 而非等效环境变量

背景

NPC 在不同触发场景下,触发内容会通过不同的环境变量传递:

  • Issue 描述 → CNB_ISSUE_DESCRIPTION
  • Issue/PR 评论 → CNB_COMMENT_BODY
  • PR 描述 → CNB_PULL_REQUEST_DESCRIPTION
  • PR 评审 → CNB_REVIEW_DESCRIPTION

如果 NPC 只处理了部分场景变量,就会导致某些场景下读不到触发内容。CNB 平台提供了统一的 CNB_NPC_TRIGGER_CONTENT 环境变量,会自动适配所有触发场景。

改动

  • main.py: 新增 TRIGGER_CONTENT_EQUIVALENTS 变量列表,在 NPC 事件阶段的 script 中检测是否使用了等效环境变量,若发现则给出 WARN 建议
  • README.md: 补充第 4 项检查说明

Ref #3

is using the merge method to merge intobb715cab
合并来自 auto/feat-trigger-content-check-58df63e0 的合并请求 #4

Successfully merged and closed

branch can be safely deleted
Reviewer
None yet
Assignee
None yet
Label
None yet
Participant