Skip to content

掃描掛回 PR、草稿也跑 CI,並修掉一段過期的註解 - #14

Merged
hoshimoe merged 1 commit into
mainfrom
claude/ci-minutes-coverage-49ujcd
Aug 22, 2026
Merged

掃描掛回 PR、草稿也跑 CI,並修掉一段過期的註解#14
hoshimoe merged 1 commit into
mainfrom
claude/ci-minutes-coverage-49ujcd

Conversation

@hoshimoe

@hoshimoe hoshimoe commented Aug 22, 2026

Copy link
Copy Markdown
Member

CI 方案升級之後,執行成本不再參與任何取捨。判準收斂成一句:CI 做得到的驗證,就讓它在合併前跑。

改動

  • security.yml:那道「掃描不上 PR」閘門拆掉。 先前的理由是「掃描找到的東西是修一次就結束的,在 PR 攔下與在 main 攔下代價一樣,所以省下每個 PR 數分鐘」。那句話的前半仍然成立,後半不再是收益;而代價一直都在——一個在 main 才被攔下的問題,是一個已經合併的問題

  • 草稿 PR 也跑 CIlinks job 上那條 draft == false 拿掉。草稿現在只表達「這份還不該被」,不再表達「這份還不該被」。

  • 修掉一段指著條文說自己不必照辦的註解。 security job 上方寫著「本倉庫是公開的,所以掃描與 links 同時起跑,不等它綠燈」,並聲明「兩種形狀是刻意的,不是漂移」。那段文字從 2026-08-16 起就是錯的,而且取代它的 needs: 一直都在——就在同一個 job 裡下面幾行,還帶著自己的註解。沒有跟上的只有那段文字。

    這是最不容易被查的形狀:讀到的人會以為「這裡已經對照過了」,於是沒有人去對照。

本機驗證

tools/check-links.py 綠(本倉庫只有 Markdown,沒有建置步驟,所以這一輪不加覆蓋率量測)。

CI 成本不再是判準(使用者裁示 2026-08-21,Enterprise 50k 分鐘/月),
判準收斂成一句:CI 做得到的驗證,就讓它在合併前跑。

- security.yml:那條 `if: github.event_name != 'pull_request'` 拆掉。
  2026-08-16 那個取捨的前半仍然成立(這幾種找到的東西是修一次就結束的),
  失效的是後半——省下的分鐘數值 0,而「在 main 才被攔下 = 已經合併」的
  代價一直都在。
- ci.yml 的檔頭:草稿 PR 也跑 CI。舊註解寫著「草稿不跑(下面每個 job 的
  if)」,而那個 if 十五個倉庫裡只有兩個真的有——註解描述的是一件沒有
  發生的事,而且方向是「以為更省」,所以不會有帳單也不會有紅燈讓人發現。
- security job 上方那段註解:它寫著「本倉庫是公開的,所以掃描與基礎驗證
  同時起跑,不等它綠燈」,並聲明「兩種形狀是刻意的,不是漂移;判準見
  workspace AGENTS.md §1.9.1」。那段文字從 2026-08-16 起就是錯的,而且
  取代它的 needs: 一直都在,就在同一個 job 裡下面幾行。一段指著條文說
  自己不必照辦的註解比沒有註解更糟:讀到的人會以為這裡已經對照過了。

本倉庫沒有 Go 程式,所以這一輪的覆蓋率量測不適用。

決策見 workspace decisions/infrastructure/CI-成本不再是判準.md。
@hoshimoe
hoshimoe merged commit 71af1bf into main Aug 22, 2026
7 checks passed
@hoshimoe
hoshimoe deleted the claude/ci-minutes-coverage-49ujcd branch August 22, 2026 11:24
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