Skip to content

refactor(build): Centralize Develocity and build cache configuration#4382

Merged
jamesarich merged 2 commits into
mainfrom
fix/build-cache
Jan 31, 2026
Merged

refactor(build): Centralize Develocity and build cache configuration#4382
jamesarich merged 2 commits into
mainfrom
fix/build-cache

Conversation

@jamesarich

Copy link
Copy Markdown
Collaborator

The Develocity and build cache settings have been extracted from the root settings.gradle.kts into a shared gradle/develocity.settings.gradle file. This new file is now applied in both the root project and the build-logic composite build.

This change centralizes the build cache logic, allowing both the main build and the build logic to share the same remote cache configuration. The property lookup for cache credentials has also been improved.

@github-actions github-actions Bot added the bugfix PR tag label Jan 30, 2026
The Develocity and build cache settings have been extracted from the root `settings.gradle.kts` into a shared `gradle/develocity.settings.gradle` file. This new file is now applied in both the root project and the `build-logic` composite build.

This change centralizes the build cache logic, allowing both the main build and the build logic to share the same remote cache configuration. The property lookup for cache credentials has also been improved.

Signed-off-by: James Rich <2199651+jamesarich@users.noreply.github.com>
This change updates the GitHub Actions workflows and Develocity configuration to support authenticated access to the Gradle remote build cache.

The reusable workflows (`reusable-lint`, `reusable-android-build`, `reusable-android-test`, and `release`) are updated to accept `GRADLE_CACHE_USERNAME` and `GRADLE_CACHE_PASSWORD` secrets.

The `develocity.settings.gradle` file is modified to only enable `push` to the remote cache if both username and password credentials are provided. This prevents "403 Forbidden" errors in CI environments that have read-only access to the cache.

Signed-off-by: James Rich <2199651+jamesarich@users.noreply.github.com>
@codecov

codecov Bot commented Jan 30, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 0.00%. Comparing base (c358081) to head (7d789d2).
⚠️ Report is 4 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@          Coverage Diff          @@
##            main   #4382   +/-   ##
=====================================
  Coverage   0.00%   0.00%           
=====================================
  Files          3       3           
  Lines         28      28           
  Branches       8       8           
=====================================
  Misses        28      28           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@jamesarich jamesarich enabled auto-merge January 30, 2026 23:18
@jamesarich jamesarich added this pull request to the merge queue Jan 30, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jan 30, 2026
@jamesarich jamesarich merged commit 1abcb4d into main Jan 31, 2026
12 of 15 checks passed
@jamesarich jamesarich deleted the fix/build-cache branch January 31, 2026 00:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix PR tag

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant