Skip to content

ci: run the release smoke test on every pull request. - #16

Merged
halkn merged 1 commit into
mainfrom
ci/shared-smoke-test
Oct 2, 2026
Merged

halkn merged 1 commit into
mainfrom
ci/shared-smoke-test

Conversation

@halkn

@halkn halkn commented Oct 2, 2026

Copy link
Copy Markdown
Owner

変更理由

v0.10.0 の release workflow が、スモークテストで廃止済みのフラグ(--no-pager --plain)を使っていたために失敗した(#15 で修正済み)。スモークテストは release.yml にしか無く、タグを push するまで実行されないため、PR の段階では気付けなかった。

変更内容

  • スモークテストのコマンドを .github/scripts/smoke-test.sh に移した。
  • release.yml: 各 native target の release バイナリに対して、このスクリプトを実行する(中身は前と同じ)。
  • ci.yml: test ジョブ(Linux・macOS・Windows)で、cargo test の後に debug ビルドしたバイナリに対して同じスクリプトを実行する。

コマンドが 1 箇所にしか無いので、フラグを変えて壊れるときは PR の CI で落ちる。

影響範囲

  • .github/scripts/smoke-test.sh(新規)、.github/workflows/ci.yml、.github/workflows/release.yml
  • CI の所要時間は、debug ビルド 1 回分だけ増える(cargo test がビルドした成果物を再利用するので、ほぼ増えない想定)。

確認手順

  • 手元で cargo build --locked → bash .github/scripts/smoke-test.sh target/debug/mdvu が exit 0 になった。
  • この PR の CI で、3 OS とも新しいステップが通ることを確認する。Windows では Git Bash が target/debug/mdvu を mdvu.exe として起動できることもここで確かめる。
  • release.yml の変更は、次のタグを push するまで実行されない。

@halkn
halkn merged commit e6f716f into main Oct 2, 2026
4 checks passed
@halkn
halkn deleted the ci/shared-smoke-test branch October 2, 2026 12:17
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