Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Configuration Cache] Disable Configuration Cache on CI #3547

Draft
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

ParaskP7
Copy link
Contributor

@ParaskP7 ParaskP7 commented Feb 5, 2025

Description

TODO


Testing Instructions

TODO

Job BT (With) BT (Without) BT (Diff) DR - Conf (With) DR - Exec (Without) DR (Diff) TE - Time [Factor] (With) TE - Time [Factor] (Without) TE (Diff)
Assemble Release 4m 46s 4m 5s -41s (✅) 7s 9s +2s (❌) 2m 0s [7.2x] 1m 58s [6.49x] -2s (✅)
Mobile App 5m 9s 4m 38s -31s (✅) 7s 8s +1s (❌) 2m 17s [7.53x] 2m 10s [6.78x] -7s (✅)
Lint 4m 57s 4m 51s -6s (✅) 8s 10s +2s (❌) 1m 45s [6.18x] 1m 51s [5.77s] +6s (❌)
Unit Tests 4m 17s 3m 39s -38s (✅) 8s 10s +2s (❌) 1m 47s [6.17x] 1m 42s [6.19x] -5s (✅)
UI Tests 8m 34s 8m 3s -31s (✅) 10s 12s +2s (❌) 2m 45s [9.68x] 2m 42s [9.32x] -3s (✅)

Note

  • Without: Without Configuration Cache
  • With: With Configuration Cache
  • BT: Build Time
  • DR: Dependency Resolution -
    • Conf: During configuration
    • Exec: During execution
  • TE: Task Execution
    • Time [Factor]: Time spent executing tasks [Serial execution factor]
Assemble BT (With CC ) BT (Without CC) BT (Diff)
Without BC (Without DC) 19m 57s 17m 29s -2m 26s (✅)
Without BC (With DC) 16m 48s 15m 40s -1m 8s (✅)
With BC (Without DC) 7m 45s 6m 49s -56s (✅)
With BC (With DC) 5m 26s 4m 58s -28s (✅)

Note

  • CC: Configuration Cache
  • BC: Build Cache
  • DC: Dependency Cache

Checklist

  • If this is a user-facing change, I have added an entry in CHANGELOG.md
  • Ensure the linter passes (./gradlew spotlessApply to automatically apply formatting/linting)
  • I have considered whether it makes sense to add tests for my changes
  • All strings that need to be localized are in modules/services/localization/src/main/res/values/strings.xml
  • Any jetpack compose components I added or changed are covered by compose previews
  • I have updated (or requested that someone edit) the spreadsheet to reflect any new or changed analytics.

I have tested any UI changes... N/A

@ParaskP7 ParaskP7 added [Type] Tooling Related to the Gradle build scripts and the setup or maintenance of the project build process. do not merge [Type] Core labels Feb 5, 2025
@dangermattic
Copy link
Collaborator

1 Message
📖 This PR is still a Draft: some checks will be skipped.

Generated by 🚫 Danger

@wpmobilebot
Copy link
Collaborator

wpmobilebot commented Feb 5, 2025

📲 You can test the changes from this Pull Request in 📱 Mobile by scanning the QR code below to install the corresponding build.
App Name 📱 Mobile
Build TypedebugProd
Commitd041cb5
Direct Downloadpocketcasts-app-prototype-build-pr3547-d041cb5.apk
📲 You can test the changes from this Pull Request in 🚗 Automotive by scanning the QR code below to install the corresponding build.
App Name 🚗 Automotive
Build TypedebugProd
Commitd041cb5
Direct Downloadpocketcasts-automotive-prototype-build-pr3547-d041cb5.apk
📲 You can test the changes from this Pull Request in ⌚ Wear by scanning the QR code below to install the corresponding build.
App Name ⌚ Wear
Build TypedebugProd
Commitd041cb5
Direct Downloadpocketcasts-wear-prototype-build-pr3547-d041cb5.apk

@ParaskP7 ParaskP7 changed the title [Configuration Cache] Disable Configuration Cache on CI#13459 [Configuration Cache] Disable Configuration Cache on CI Feb 5, 2025
@ParaskP7 ParaskP7 force-pushed the ci/disable-configuration-cache-on-ci branch from aeef473 to dafe832 Compare February 5, 2025 13:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do not merge [Type] Core [Type] Tooling Related to the Gradle build scripts and the setup or maintenance of the project build process.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants