Skip to content

CI安定化 - #191

Merged
riiimparm merged 1 commit into
developfrom
fix/ci-pnpm-frozen-install
Sep 1, 2026
Merged

CI安定化#191
riiimparm merged 1 commit into
developfrom
fix/ci-pnpm-frozen-install

Conversation

@riiimparm

@riiimparm riiimparm commented Aug 29, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • pnpm/action-setupversion: 8にハードコードされていたが、pnpm-lock.yaml9.0だったので不一致を解消
  • pnpm install--frozen-lockfileつきで実行するよう変更。
  • 同一PRへの追加pushで前のワークフロー実行が自動キャンセルされるように

pnpm/action-setupがversion:8にハードコードされていたが、pnpm-lock.yamlの
lockfileVersionは9.0で本来pnpm9+が必要な状態だった。バージョン不一致により
CIはlockfileを正しく解釈できず依存関係を独自に再解決しており、これが
prettier等のバージョンがローカルとCIで食い違いformat:checkが不安定に
失敗する原因になっていた。

package.jsonにpackageManagerを追加してpnpmバージョンを自動検出させ、
pnpm installはフローズンロックファイルで実行するよう変更した。
あわせてconcurrencyグループを追加し、同一PRへの追加pushで古い実行が
自動キャンセルされるようにした。
@riiimparm riiimparm self-assigned this Aug 29, 2026
@riiimparm riiimparm changed the title 🔧 build: CIのpnpmインストールを決定的にする 🔧 build: CI安定化 Aug 29, 2026
@riiimparm riiimparm changed the title 🔧 build: CI安定化 CI安定化 Aug 29, 2026
@riiimparm
riiimparm requested a review from naoki-00-ito August 29, 2026 08:30
Comment thread .github/workflows/test.yml

@naoki-00-ito naoki-00-ito left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@riiimparm
LGTM!

@riiimparm
riiimparm merged commit c96c57d into develop Sep 1, 2026
1 check passed
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.

2 participants