-
Notifications
You must be signed in to change notification settings - Fork 0
Improve conan package caching in CI #314
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR improves Conan package caching in CI by implementing custom cache management instead of relying on the conan-io/setup-conan action. The changes introduce manual cache restoration and saving with intelligent cleanup of unused packages to optimize cache size and efficiency.
Key Changes:
- Replaced
conan-io/setup-conanwith custom cache management using GitHub Actions cache - Added cache cleanup step to remove unused Conan recipes and binaries based on LRU (1 hour)
- Reorganized workflow to set up mise tools earlier and added finalization step for cache saving
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
.github/workflows/cpp-build-test-run.yaml |
Reorganized workflow steps to set up mise earlier, added conan-cache-key-prefix input, and introduced cmake-project-finalize step |
.github/actions/cmake-project-setup/action.yaml |
Replaced conan-io/setup-conan action with manual cache restoration, added Conan profile detection, and exposed cache-matched-key output |
.github/actions/cmake-project-setup/init.sh |
New script to initialize CONAN_HOME environment variable and output cache path |
.github/actions/cmake-project-finalize/action.yaml |
New action to finalize and save Conan cache with conditional logic based on package hash changes |
.github/actions/cmake-project-finalize/init.sh |
New script to clean up unused Conan packages, generate package hash, and determine if cache should be saved |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
e917cc6 to
3982249
Compare
3982249 to
91f8c94
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
91f8c94 to
c3dd199
Compare
c3dd199 to
39c21b9
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
8a85afb to
8572ff6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
8572ff6 to
6197569
Compare
No description provided.