Skip to content

Enable Gradle configuration and build caches - #6

Merged
fanis merged 2 commits into
masterfrom
claude/project-review-optimization-4xcgkm
Jul 3, 2026
Merged

Enable Gradle configuration and build caches#6
fanis merged 2 commits into
masterfrom
claude/project-review-optimization-4xcgkm

Conversation

@fanis

@fanis fanis commented Jul 3, 2026

Copy link
Copy Markdown
Owner

Follows up on the configuration-cache suggestion in the v1.1.0 Cut Release run:

  • Enable org.gradle.configuration-cache and org.gradle.caching in gradle.properties (verified locally: cache stored, reused, warm test run in under a second).
  • Cut Release now runs gradle test assembleRelease as a single invocation — one configuration phase and daemon start instead of two, and a failing test still aborts the release before anything is pushed.
  • The Cut Release job deliberately never persists its configuration cache to the Actions cache (it embeds the signing config); it is rebuilt on each run.
  • The PR test workflow accepts an optional GRADLE_ENCRYPTION_KEY secret (e.g. openssl rand -base64 16) to persist its configuration cache across runs — safe there because test runs have no keystore, so the cache contains no signing data. Without the secret nothing is persisted; the in-run caching still applies.

claude added 2 commits July 3, 2026 20:37
Turn on org.gradle.configuration-cache and org.gradle.caching, run the Cut
Release tests and signed build in one Gradle invocation (one configuration
phase, one daemon), and pass an optional GRADLE_ENCRYPTION_KEY to
setup-gradle so the configuration cache can persist across CI runs when the
secret is configured.
The Cut Release configuration cache embeds the signing config, so drop the
cache-encryption-key from that workflow entirely; it is rebuilt each run.
Keep the optional key on the PR test workflow, whose runs have no keystore
and therefore no signing data in the cache.
@fanis
fanis merged commit 7d147fe into master Jul 3, 2026
1 check passed
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.

2 participants