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

Get rid of Symbols synchronization on Usages, Builders feaures #190

Merged
merged 3 commits into from
Oct 10, 2023

Conversation

lehvolk
Copy link
Collaborator

@lehvolk lehvolk commented Oct 10, 2023

Apart from that

  • drop support of PosgreSQL
  • add ability to write DB migrations (aka refactorings)

@github-actions
Copy link
Contributor

github-actions bot commented Oct 10, 2023

Lifecycle test results

46 tests  ±0   46 ✔️ ±0   2m 15s ⏱️ +11s
  4 suites ±0     0 💤 ±0 
  4 files   ±0     0 ±0 

Results for commit dff5e16. ± Comparison against base commit 099fc26.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Contributor

github-actions bot commented Oct 10, 2023

Test results on JDK 19

1 256 tests  ±0   1 245 ✔️ ±0   6m 27s ⏱️ - 1m 13s
     43 suites ±0        11 💤 ±0 
     43 files   ±0          0 ±0 

Results for commit dff5e16. ± Comparison against base commit 099fc26.

♻️ This comment has been updated with latest results.

@codecov
Copy link

codecov bot commented Oct 10, 2023

Codecov Report

Merging #190 (dff5e16) into develop (099fc26) will increase coverage by 0.25%.
The diff coverage is 71.77%.

@@              Coverage Diff              @@
##             develop     #190      +/-   ##
=============================================
+ Coverage      76.98%   77.23%   +0.25%     
- Complexity      1559     1563       +4     
=============================================
  Files            157      157              
  Lines           8784     8824      +40     
  Branches        1536     1540       +4     
=============================================
+ Hits            6762     6815      +53     
+ Misses          1426     1402      -24     
- Partials         596      607      +11     
Files Coverage Δ
...c/main/kotlin/org/jacodb/impl/features/Builders.kt 89.21% <90.90%> (+0.21%) ⬆️
...in/kotlin/org/jacodb/impl/storage/JcRefactoring.kt 90.32% <90.32%> (ø)
...n/org/jacodb/impl/storage/SQLitePersistenceImpl.kt 67.27% <25.00%> (ø)
...src/main/kotlin/org/jacodb/impl/features/Usages.kt 92.70% <76.92%> (-0.26%) ⬇️
...core/src/main/kotlin/org/jacodb/impl/JcSettings.kt 42.70% <0.00%> (-0.15%) ⬇️
...tlin/org/jacodb/impl/storage/PersistenceService.kt 90.77% <71.42%> (-2.79%) ⬇️

... and 3 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@github-actions
Copy link
Contributor

github-actions bot commented Oct 10, 2023

Test results on JDK 8

1 256 tests  ±0   1 243 ✔️ ±0   8m 1s ⏱️ +46s
     43 suites ±0        13 💤 ±0 
     43 files   ±0          0 ±0 

Results for commit dff5e16. ± Comparison against base commit 099fc26.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Contributor

github-actions bot commented Oct 10, 2023

Test results on JDK 11

1 256 tests  ±0   1 247 ✔️ ±0   7m 51s ⏱️ -29s
     43 suites ±0          9 💤 ±0 
     43 files   ±0          0 ±0 

Results for commit dff5e16. ± Comparison against base commit 099fc26.

♻️ This comment has been updated with latest results.

@lehvolk lehvolk merged commit 7eb353d into develop Oct 10, 2023
13 of 14 checks passed
@lehvolk lehvolk deleted the lehvolk/features-tread-safe branch November 1, 2023 08:30
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.

1 participant