Skip to content

Conversation

@Saloed
Copy link
Collaborator

@Saloed Saloed commented Feb 10, 2025

Pluggable caches API moved to the jacodb-storage-api module, implementations are in the jacodb-storage module. The jacodb-storage module has compileOnly dependencies on the libraries providing caches. So to use caches without dependency on the jacodb-core module, one should define particular dependency of the caching library, Guava or Xodus.

Pluggable caches API moved to the jacodb-storage-api module, implementations
are in the jacodb-storage module. The jacodb-storage module has compileOnly
dependencies on the libraries providing caches. So to use caches without
dependency on the jacodb-core module, one should define particular
dependency of the caching library, Guava or Xodus.
@github-actions
Copy link
Contributor

Lifecycle test results

78 tests  ±0   78 ✅ ±0   1m 40s ⏱️ +13s
 7 suites ±0    0 💤 ±0 
 7 files   ±0    0 ❌ ±0 

Results for commit 0b3fefd. ± Comparison against base commit f66dd8c.

@github-actions
Copy link
Contributor

Test results on JDK 19

 72 files  ±0   72 suites  ±0   4m 38s ⏱️ -1s
628 tests ±0  618 ✅ ±0  10 💤 ±0  0 ❌ ±0 
698 runs  ±0  688 ✅ ±0  10 💤 ±0  0 ❌ ±0 

Results for commit 0b3fefd. ± Comparison against base commit f66dd8c.

@github-actions
Copy link
Contributor

Test results on JDK 8

 72 files  ±0   72 suites  ±0   4m 52s ⏱️ ±0s
628 tests ±0  615 ✅ ±0  13 💤 ±0  0 ❌ ±0 
698 runs  ±0  685 ✅ ±0  13 💤 ±0  0 ❌ ±0 

Results for commit 0b3fefd. ± Comparison against base commit f66dd8c.

@codecov
Copy link

codecov bot commented Feb 10, 2025

Codecov Report

Attention: Patch coverage is 0% with 3 lines in your changes missing coverage. Please review.

Project coverage is 69.19%. Comparing base (f66dd8c) to head (0b3fefd).
Report is 1 commits behind head on neo.

Files with missing lines Patch % Lines
...org/jacodb/impl/storage/AbstractJcDbPersistence.kt 0.00% 2 Missing ⚠️
...kotlin/org/jacodb/impl/caches/guava/GuavaCaches.kt 0.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##                neo     #304      +/-   ##
============================================
- Coverage     69.43%   69.19%   -0.24%     
+ Complexity     2475     2474       -1     
============================================
  Files           271      269       -2     
  Lines         15455    15441      -14     
  Branches       2284     2281       -3     
============================================
- Hits          10731    10685      -46     
- Misses         3791     3829      +38     
+ Partials        933      927       -6     

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

@github-actions
Copy link
Contributor

Test results on JDK 11

 72 files  ±0   72 suites  ±0   5m 18s ⏱️ -1s
628 tests ±0  620 ✅ ±0  8 💤 ±0  0 ❌ ±0 
698 runs  ±0  690 ✅ ±0  8 💤 ±0  0 ❌ ±0 

Results for commit 0b3fefd. ± Comparison against base commit f66dd8c.

@Saloed Saloed merged commit 6db1ff5 into neo Feb 10, 2025
9 of 11 checks passed
@Saloed Saloed deleted the saloed/move-caches branch February 10, 2025 21:11
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