Skip to content

chore: remove required ci with java8#2503

Merged
imbajin merged 1 commit intoapache:masterfrom
Pengzna:remove_JDK8_CI
Apr 3, 2024
Merged

chore: remove required ci with java8#2503
imbajin merged 1 commit intoapache:masterfrom
Pengzna:remove_JDK8_CI

Conversation

@Pengzna
Copy link
Copy Markdown
Contributor

@Pengzna Pengzna commented Apr 3, 2024

as title

@dosubot dosubot bot added size:XS This PR changes 0-9 lines, ignoring generated files. ci-cd Build or deploy labels Apr 3, 2024
@VGalaxies VGalaxies requested a review from imbajin April 3, 2024 14:38
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Apr 3, 2024
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 53.26%. Comparing base (27cf52b) to head (babc54f).

Additional details and impacted files
@@              Coverage Diff              @@
##             master    #2503       +/-   ##
=============================================
- Coverage     66.31%   53.26%   -13.05%     
+ Complexity      829      545      -284     
=============================================
  Files           511      511               
  Lines         42649    42649               
  Branches       5942     5942               
=============================================
- Hits          28281    22717     -5564     
- Misses        11557    17489     +5932     
+ Partials       2811     2443      -368     

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

@imbajin imbajin merged commit 483bca5 into apache:master Apr 3, 2024
@imbajin imbajin changed the title Remove JDK8 required CI for further version(pd-store migration) chore: remove required ci with java8 Apr 3, 2024
VGalaxies pushed a commit that referenced this pull request Apr 3, 2024
@Pengzna Pengzna deleted the remove_JDK8_CI branch April 8, 2024 07:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-cd Build or deploy lgtm This PR has been approved by a maintainer size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants