Skip to content

Append build flavor to the cache#198

Draft
shaosu-nvidia wants to merge 1 commit intomainfrom
ssx/oakd_build_time
Draft

Append build flavor to the cache#198
shaosu-nvidia wants to merge 1 commit intomainfrom
ssx/oakd_build_time

Conversation

@shaosu-nvidia
Copy link
Contributor

@shaosu-nvidia shaosu-nvidia commented Mar 2, 2026

Summary by CodeRabbit

  • Chores
    • Optimized Hunter dependency caching in CI/CD pipelines to maintain separate caches for Debug and Release builds, improving build efficiency and reliability.

@shaosu-nvidia shaosu-nvidia marked this pull request as draft March 2, 2026 22:10
@coderabbitai
Copy link

coderabbitai bot commented Mar 2, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 3dd2f87 and 09cb8fb.

📒 Files selected for processing (2)
  • .github/workflows/build-ubuntu.yml
  • .github/workflows/build-windows.yml

📝 Walkthrough

Walkthrough

GitHub Actions workflows for Ubuntu and Windows builds have been updated to include the build type (Debug/Release) in Hunter package cache keys, enabling separate caching per build configuration instead of shared caching across all build types.

Changes

Cohort / File(s) Summary
Hunter Cache Configuration
.github/workflows/build-ubuntu.yml, .github/workflows/build-windows.yml
Updated Hunter cache key and restore-keys to include matrix.build_type, enabling per-build-type (Debug/Release) caching for Hunter dependencies.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A cache so clever, split by build,
Debug and Release, separately hulled,
No more mixing what shouldn't be mixed,
Hunter's packages perfectly fixed! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Append build flavor to the cache' directly and specifically describes the main change: adding matrix.build_type to Hunter cache keys for Debug/Release-specific caching.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch ssx/oakd_build_time

Comment @coderabbitai help to get the list of available commands and usage tips.

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