Skip to content

Conversation

@williscool
Copy link
Owner

@williscool williscool commented Jan 19, 2026

Note

Improves CI caching correctness and alignment across workflows.

  • Include yarn.lock in Android build outputs cache keys in common-setup and cache-update to invalidate when JS deps change (impacts React Native codegen)
  • Clarify that unit tests restore but do not save build outputs and that codegen cannot be skipped due to Gradle task graph
  • Align cache-warming workflow with main CI by defining matching CCACHE_* environment variables

Written by Cursor Bugbot for commit ce19fd1. This will update automatically on new commits. Configure here.

The main workflow sets CCACHE_BASEDIR which changes how paths are hashed.
Without matching env vars in the cache warmer, ccache gets 0% hit rate
because the path hashes don't match between workflows.

This was causing release builds to take 22+ min instead of ~10 min.

Co-authored-by: wharris <wharris@upscalews.com>
@cursor cursor bot force-pushed the cursor/unit-tests-codegen-cache branch from ce19fd1 to ffca75a Compare January 19, 2026 02:02
@williscool williscool enabled auto-merge (squash) January 19, 2026 02:04
@williscool williscool disabled auto-merge January 19, 2026 02:04
@williscool williscool merged commit f847aff into master Jan 19, 2026
6 checks passed
@github-actions
Copy link

Build artifacts for PR #211 (commit cfaf7ec) are available:

You can download these artifacts from the "Artifacts" section of the workflow run.

@github-actions
Copy link

Build artifacts for PR #211 (commit 4bbd664) are available:

You can download these artifacts from the "Artifacts" section of the workflow run.

@github-actions
Copy link

Code Coverage Report

Overall Project 26.37%

There is no coverage information present for the Files changed

@github-actions
Copy link

📊 Code Coverage Summary

Coverage Type Coverage
Overall 26.37
Changed Files 100

View detailed coverage report

@github-actions
Copy link

📊 Code Coverage Summary

Coverage Type Coverage
Overall 26.37
Changed Files 100

View detailed coverage report

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.

3 participants