ci: cache ptoas binary and extract version to env var#799
ci: cache ptoas binary and extract version to env var#799luohuan19 wants to merge 1 commit intohw-native-sys:mainfrom
Conversation
|
Note Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported. |
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
✅ Files skipped from review due to trivial changes (1)
📝 WalkthroughWalkthroughCI workflow updated to add a PTOAS version env var, cache the ptoas binary, conditionally skip installation on cache hit, and rename "simpler" clone/install step labels from "(pinned)" to "(stable)". Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
- Add `actions/cache@v4` step to cache the ptoas binary keyed by version and architecture, skipping download on cache hit - Extract PTOAS_VERSION to a job-level env var (was inline in the install script) to serve as the cache key - Rename "Clone and install simpler repository (pinned)" step label to "(stable)" to match the actual branch name being cloned
actions/cache@v4step to cache the ptoas binary keyed by version and architecture, skipping download on cache hit