Add a push/PR CI workflow #7

Closed
opened 2026-07-28 12:21:10 +09:00 by FLYEMOJ1 · 0 comments
FLYEMOJ1 commented 2026-07-28 12:21:10 +09:00 (Migrated from github.com)

问题

.github/workflows/ 里只有 release.yml,触发条件是 on: push: tags: ['v*']。没有任何 workflow 在 push 或 PR 上跑。

也就是说 Go 测试、go vet、panel 的 vitest 全都只在本地跑过。一个能编译但测试挂了的改动可以合进 main,一路到打 tag 才第一次被 CI 碰到 —— 而那时候它已经在发布路径上了。

panel 的 vitest 尤其没人跑过:release.yml 走 Dockerfile,Dockerfile 跑的是 npm run build,不是 npm test。

证据

Identified by Claude Opus 5 (claude-opus-5) while auditing the CI surface on 2026-07-28.

  • .github/workflows/ — 目录下只有 release.yml 一个文件
  • .github/workflows/release.yml:22-24:
on:
  push:
    tags: ['v*']

验收

开一个 PR,能看到 Go 测试和 panel 测试各自跑起来并报状态。

备注

Felis-Legacy 那边已经有一个 ci.yml 在 push/PR 上跑(虽然覆盖面偏窄),可以照着它的骨架起。

## 问题 `.github/workflows/` 里只有 `release.yml`,触发条件是 `on: push: tags: ['v*']`。没有任何 workflow 在 push 或 PR 上跑。 也就是说 Go 测试、`go vet`、panel 的 vitest 全都只在本地跑过。一个能编译但测试挂了的改动可以合进 main,一路到打 tag 才第一次被 CI 碰到 —— 而那时候它已经在发布路径上了。 panel 的 vitest 尤其没人跑过:release.yml 走 Dockerfile,Dockerfile 跑的是 `npm run build`,不是 `npm test`。 ## 证据 Identified by Claude Opus 5 (claude-opus-5) while auditing the CI surface on 2026-07-28. - `.github/workflows/` — 目录下只有 `release.yml` 一个文件 - `.github/workflows/release.yml:22-24`: ```yaml on: push: tags: ['v*'] ``` ## 验收 开一个 PR,能看到 Go 测试和 panel 测试各自跑起来并报状态。 ## 备注 Felis-Legacy 那边已经有一个 `ci.yml` 在 push/PR 上跑(虽然覆盖面偏窄),可以照着它的骨架起。
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: FelisMC/Felis#7