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

ref: Consolidate CacheVersions #1041

Merged
merged 4 commits into from
Feb 15, 2023
Merged

ref: Consolidate CacheVersions #1041

merged 4 commits into from
Feb 15, 2023

Conversation

Swatinem
Copy link
Member

Moves all the places that define CacheVersions into a central place, and makes it mandatory to specify explicit versions.

It also bumps some cache versions, to start lazy recomputation of some of our caches to the new CacheKey format, specifically: Bitcode, Il2cpp, PortablePDB, and Objects.
CFI, SymCache and Meta caches in turn depend on other categories, and will be bumped once the first categories have been migrated.

@Swatinem Swatinem requested a review from a team February 15, 2023 13:42
Moves all the places that define `CacheVersions` into a central place, and makes it mandatory to specify
explicit versions.

It also bumps some cache versions, to start lazy recomputation of some of our caches to the new `CacheKey` format, specifically:
Bitcode, Il2cpp, and Objects.
CFI, SymCache, Portable PDB, and Meta caches in turn depend on other categories, and will be bumped once the first categories
have been migrated.
@codecov
Copy link

codecov bot commented Feb 15, 2023

Codecov Report

Merging #1041 (9b57d63) into master (445e67b) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1041      +/-   ##
==========================================
- Coverage   75.12%   75.12%   -0.01%     
==========================================
  Files          84       84              
  Lines       12006    12004       -2     
==========================================
- Hits         9020     9018       -2     
  Misses       2986     2986              

@Swatinem Swatinem merged commit 3b329eb into master Feb 15, 2023
@Swatinem Swatinem deleted the ref/versions branch February 15, 2023 14:17
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