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

Use Temurin 11 JDK for CI #190

Merged
merged 2 commits into from
Jul 5, 2022
Merged

Use Temurin 11 JDK for CI #190

merged 2 commits into from
Jul 5, 2022

Conversation

twyatt
Copy link
Member

@twyatt twyatt commented Jun 30, 2022

NOTE: Adopt OpenJDK got moved to Eclipse Temurin and won't be updated anymore. It is highly recommended to migrate workflows from adopt to temurin to keep receiving software and security updates. See more details in the Good-bye AdoptOpenJDK post.

https://github.com/actions/setup-java#supported-distributions

@twyatt twyatt requested review from a team, QuantumRand and cedrickcooke June 30, 2022 21:40
@twyatt twyatt enabled auto-merge (squash) June 30, 2022 21:40
@twyatt twyatt added the maintenance General maintenance that doesn't effect the public API. label Jun 30, 2022
@github-actions
Copy link

github-actions bot commented Jun 30, 2022

Unit Test Results

  32 files    32 suites   8s ⏱️
201 tests 194 ✔️   7 💤 0
218 runs  206 ✔️ 12 💤 0

Results for commit 442499a.

♻️ This comment has been updated with latest results.

@codecov
Copy link

codecov bot commented Jun 30, 2022

Codecov Report

Merging #190 (442499a) into main (1c1547e) will decrease coverage by 8.98%.
The diff coverage is 80.27%.

@@             Coverage Diff              @@
##               main     #190      +/-   ##
============================================
- Coverage     94.44%   85.46%   -8.99%     
- Complexity       51       91      +40     
============================================
  Files            13       20       +7     
  Lines           126      282     +156     
  Branches          8       15       +7     
============================================
+ Hits            119      241     +122     
- Misses            6       37      +31     
- Partials          1        4       +3     
Impacted Files Coverage Δ
coroutines/src/commonMain/kotlin/flow/Flows.kt 0.00% <ø> (ø)
logging/src/commonMain/kotlin/Metadata.kt 76.92% <0.00%> (-0.86%) ⬇️
logging/src/commonMain/kotlin/Sensitivity.kt 0.00% <0.00%> (ø)
...tines/src/commonMain/kotlin/delay/DelayStrategy.kt 47.50% <47.50%> (ø)
logging/src/commonMain/kotlin/DispatchLogger.kt 92.10% <89.65%> (-7.90%) ⬇️
logging/src/commonMain/kotlin/Log.kt 98.27% <93.75%> (-1.73%) ⬇️
logging/src/commonMain/kotlin/FilterLogger.kt 95.65% <95.65%> (ø)
...t/src/commonMain/kotlin/TuulboxKtorClientLogger.kt 100.00% <100.00%> (ø)
logging/src/commonMain/kotlin/FilterLevelLogger.kt 100.00% <100.00%> (ø)
logging/src/commonMain/kotlin/LogLevel.kt 100.00% <100.00%> (ø)
... and 7 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8ec61aa...442499a. Read the comment docs.

@twyatt

This comment was marked as resolved.

@twyatt twyatt merged commit 6fb4f31 into main Jul 5, 2022
@twyatt twyatt deleted the twyatt/ci/temurin11 branch July 5, 2022 18:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance General maintenance that doesn't effect the public API.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants