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.

Sign up for GitHub

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

Jump to bottom

Activate Develocity's Build Cache #22523

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Activate Develocity's Build Cache #22523

wants to merge 6 commits into from
+16 −12

Conversation

adpi2
Copy link
Member

@adpi2 adpi2 commented Feb 5, 2025 •
edited
Loading

  1. Enable read and write to local cache for all users
  2. Enable read from remote cache for all users
  3. Enable write to remote cache on CI only
  4. Disable test caching on all projects: We generally don't rerun tests if nothing changed. When users do run the tests locally, they probably want to see their executions.

ping @hamzaremmal

@tgodzik
Copy link
Contributor

tgodzik commented Feb 5, 2025

Och, awesome! Will we be able to use that locally?

@adpi2
Copy link
Member Author

adpi2 commented Feb 5, 2025

Och, awesome! Will we be able to use that locally?

Yes, once this is merged you will be able to download compilation outputs from the remote server without any further configuration.

@adpi2
Copy link
Member Author

adpi2 commented Feb 21, 2025

Superseded by #22578

@adpi2 adpi2 reopened this Feb 26, 2025
@adpi2 adpi2 marked this pull request as ready for review February 26, 2025 10:17
@hamzaremmal hamzaremmal enabled auto-merge (squash) February 26, 2025 10:17
@adpi2
Copy link
Member Author

adpi2 commented Feb 26, 2025 •
edited
Loading

I rebased the branch because Mima job failed: https://github.com/scala/scala3/actions/runs/13541557343

@adpi2
Copy link
Member Author

adpi2 commented Feb 26, 2025

Mima failed again: https://github.com/scala/scala3/actions/runs/13544643262/job/37853234696

I suspect there is an incorrect cache hit. We read something from the cache instead of recompiling it and then we get a TASTy version mismatch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hamzaremmal hamzaremmal hamzaremmal approved these changes

Assignees

@adpi2 adpi2

@hamzaremmal hamzaremmal

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@adpi2 @tgodzik @hamzaremmal