Skip to content

fix: org-gate hygiene 重复 uses 键(required workflow 启动失败根因) - #24

Merged
randypanding merged 1 commit into
mainfrom
p3-1-org-gate-dupfix
Aug 20, 2026
Merged

fix: org-gate hygiene 重复 uses 键(required workflow 启动失败根因)#24
randypanding merged 1 commit into
mainfrom
p3-1-org-gate-dupfix

Conversation

@randypanding

@randypanding randypanding commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

ADR-0046 配套修复(P3-1,.github#95)。

探针实证(Script_Writer run 32345359860):required workflow 机制端到端可用——启动失败注解直接指向中心仓文件(审判源=中心仓 main 的直接证据)。根因:#23 插入 job 守卫时 hygiene job 产生重复 uses: 键(PyYAML 容忍重复键故常规校验未拦;Actions 引擎拒绝)。修复并以严格重复键语义校验通过。副产物教训:建议本仓自测补 actionlint(可后续)。

@coderabbitai

coderabbitai Bot commented Aug 20, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: dcb9e863-0c2f-46d8-a7f8-a60cbd8622a7

📥 Commits

Reviewing files that changed from the base of the PR and between ae7079d and beecc73.

📒 Files selected for processing (1)
  • .github/workflows/org-gate.yml
💤 Files with no reviewable changes (1)
  • .github/workflows/org-gate.yml

Included review availability: Your plan provides up to 10 included reviews per hour; 2 remain after this review.


📝 Walkthrough

Walkthrough

本次变更移除 hygiene job 中的空 uses: 配置,并保留固定 commit 的有效 workflow 引用。

Changes

hygiene 工作流修正

Layer / File(s) Summary
移除空的 workflow 配置
.github/workflows/org-gate.yml
删除条件之后的空 uses: 键。保留后续指向固定 commit 的有效 workflow 引用。

Possibly related PRs

Suggested labels: security, bug

🚥 Pre-merge checks | ✅ 1 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Title check ⚠️ Warning 标题使用了有效的 fix: 前缀,且准确描述了修复内容,但长度为 57 个字符,超过 50 个字符限制。 将标题缩短至 50 个字符以内,同时保留 fix: 前缀和对 org-gate hygiene 重复 uses: 问题的准确描述。
✅ Passed checks (1 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch p3-1-org-gate-dupfix

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

@qodo-code-review

Copy link
Copy Markdown

PR Summary by Qodo

Remove duplicate uses key from org-gate hygiene job

🐞 Bug fix ⚙️ Configuration changes 🕐 Less than 10 minutes

Grey Divider

AI Description

• Removes the empty duplicate uses key from the org-gate hygiene job.
• Restores GitHub Actions parsing so the required reusable workflow can start.
High-Level Assessment

Removing the stray key is the minimal and correct fix because the remaining pinned uses declaration already defines the intended reusable workflow. Rewriting the job or changing YAML tooling would add unnecessary scope; strict duplicate-key linting can be addressed separately.

Files changed (1) +0 / -1

Bug fix (1) +0 / -1
org-gate.ymlRemove duplicate hygiene workflow uses key +0/-1

Remove duplicate hygiene workflow uses key

• Deletes the empty duplicate 'uses' mapping key that caused GitHub Actions to reject the workflow. The hygiene job now retains only its valid, commit-pinned reusable workflow reference.

.github/workflows/org-gate.yml

@coderabbitai coderabbitai Bot added bug Something isn't working security labels Aug 20, 2026
@qodo-code-review

Copy link
Copy Markdown

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

Tip of the day
💡 Did you know, you can copy the agent prompt from any finding and feed it to your IDE agent

More tips ↗ | Customize Qodo ↗ | Qodo docs ↗

Grey Divider

Qodo Logo

@randypanding
randypanding merged commit 7fb0eaf into main Aug 20, 2026
17 of 19 checks passed
@randypanding
randypanding deleted the p3-1-org-gate-dupfix branch August 20, 2026 08:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working security

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant