Skip to content

build(ci): optimize release workflow and update Room configuration#4775

Merged
jamesarich merged 2 commits intomainfrom
fix/aboutlibs
Mar 13, 2026
Merged

build(ci): optimize release workflow and update Room configuration#4775
jamesarich merged 2 commits intomainfrom
fix/aboutlibs

Conversation

@jamesarich
Copy link
Copy Markdown
Collaborator

This commit refactors the CI release workflow to improve reliability and cleans up redundant Room compiler configurations in the build logic.

Specific changes include:

  • CI/CD (GitHub Actions):
    • Updated build provenance attestation steps to run only on success() instead of always(), preventing attestation attempts for failed builds.
    • Removed redundant GITHUB_TOKEN environment variables from the exportLibraryDefinitions steps across Google Play, F-Droid, and Native distribution jobs.
  • Build Logic:
    • Removed kspCommonMainMetadata dependency for the Room compiler in AndroidRoomConventionPlugin.kt, streamlining the KSP configuration for Android.

This commit refactors the CI release workflow to improve reliability and cleans up redundant Room compiler configurations in the build logic.

Specific changes include:
- **CI/CD (GitHub Actions):**
    - Updated build provenance attestation steps to run only on `success()` instead of `always()`, preventing attestation attempts for failed builds.
    - Removed redundant `GITHUB_TOKEN` environment variables from the `exportLibraryDefinitions` steps across Google Play, F-Droid, and Native distribution jobs.
- **Build Logic:**
    - Removed `kspCommonMainMetadata` dependency for the Room compiler in `AndroidRoomConventionPlugin.kt`, streamlining the KSP configuration for Android.

Signed-off-by: James Rich <[email protected]>
@github-actions github-actions bot added the bugfix PR tag label Mar 13, 2026
@jamesarich jamesarich merged commit 0ed9b66 into main Mar 13, 2026
2 checks passed
@jamesarich jamesarich deleted the fix/aboutlibs branch March 13, 2026 03:46
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 13, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 9.28%. Comparing base (5a548d2) to head (0b9a99d).
⚠️ Report is 5 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff            @@
##            main   #4775      +/-   ##
========================================
- Coverage   9.51%   9.28%   -0.24%     
========================================
  Files        480     516      +36     
  Lines      16372   17240     +868     
  Branches    2401    2580     +179     
========================================
+ Hits        1558    1600      +42     
- Misses     14555   15381     +826     
  Partials     259     259              

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

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