Repository navigation
CI の Rust の検査を、apps/desktop か ci.yml が変わった PR だけで動かす - #3
Merged
Merged
Conversation
Tauri の Linux 向けライブラリの取得 (apt-get install) に時間がかかり、Rust に触れない PR でも毎回待たされていた。 変更の検出のジョブ (changes) が PR の差分を見て、apps/desktop/** か ci.yml が変わったときだけ Rust のジョブを動かす。 main への push では常に動かし、飛ばした PR のあとでも Rust 側の壊れに気づけるようにする。 Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01EdDb33LK7o1arFRw9rP37g
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Deploying untie with
|
| Latest commit: |
5dd5cf6
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://f45162ae.untie-3lp.pages.dev |
| Branch Preview URL: | https://ci-rust-only-when-desktop-ch.untie-3lp.pages.dev |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
何を変えたか
Tauri の Linux 向けライブラリの取得 (
apt-get install) に時間がかかり、Rust に触れない PR でも毎回それを待たされていました。Rust の検査 (cargo check) を、Rust 側に関係する変更があるときだけ動かします。changesを足す。PR の差分 (base.shaと PR のマージ結果) を見て、apps/desktop/**(src-tauriとCargo.lockを含む) か.github/workflows/ci.ymlが変わったときだけrust=trueを出すRust (cargo check)はneeds: changesとif: needs.changes.outputs.rust == 'true'で、rust=trueのときだけ動かす。ジョブ名は変えていないので、必須のチェックにしていても、飛ばしたときは成功として扱われるmainへの push では常に動かす。飛ばした PR のあとでも、Rust 側の壊れにmainで気づけるdocs/design.mdの CI の説明を合わせる確認
mainと PR 情報の置き場所と画面の情報を操作から導くようにする (モデルの形式 0.4.0) #2 (Rust に触れない) の差 →rust=falseapps/desktopに触れた過去のコミット (93c695e) →rust=trueci.ymlを変更) →rust=trueci.ymlを変えているので、Rust のジョブも動く。新しい構成でも Rust のジョブが通ることの確認になる (Actions の動きそのものは、この PR の CI で確かめられる)知っておいてほしいこと
mainの push で動くapt-get install自体は、速くなっていない (今回は、動かす回数を減らした)Generated by Claude Code