Skip to content

upgrade gradle to 8.1.1#893

Merged
sbcd90 merged 2 commits intoopensearch-project:mainfrom
sbcd90:3.0_gradle
Apr 28, 2023
Merged

upgrade gradle to 8.1.1#893
sbcd90 merged 2 commits intoopensearch-project:mainfrom
sbcd90:3.0_gradle

Conversation

@sbcd90
Copy link
Copy Markdown
Contributor

@sbcd90 sbcd90 commented Apr 28, 2023

Issue #, if available:

Description of changes:
upgrade gradle to 8.1.1
ref: opensearch-project/OpenSearch#7270

CheckList:
[X] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Signed-off-by: Subhobrata Dey <sbcd90@gmail.com>
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Apr 28, 2023

Codecov Report

Merging #893 (1801fde) into main (37b81fa) will increase coverage by 0.04%.
The diff coverage is 50.00%.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@             Coverage Diff              @@
##               main     #893      +/-   ##
============================================
+ Coverage     75.76%   75.81%   +0.04%     
+ Complexity      111      110       -1     
============================================
  Files           125      125              
  Lines          6908     6908              
  Branches       1037     1037              
============================================
+ Hits           5234     5237       +3     
+ Misses         1143     1141       -2     
+ Partials        531      530       -1     
Impacted Files Coverage Δ
...lin/org/opensearch/alerting/alerts/AlertIndices.kt 74.07% <50.00%> (ø)
.../kotlin/org/opensearch/alerting/util/IndexUtils.kt 77.96% <50.00%> (ø)

... and 3 files with indirect coverage changes

Comment thread build-tools/merged-coverage.gradle Outdated
}

task jacocoMerge(type: JacocoMerge) {
/*task jacocoMerge(type: JacocoMerge) {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why are we commenting this?

Copy link
Copy Markdown
Contributor Author

@sbcd90 sbcd90 Apr 28, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Signed-off-by: Subhobrata Dey <sbcd90@gmail.com>
@sbcd90 sbcd90 merged commit 58d2b03 into opensearch-project:main Apr 28, 2023
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.

4 participants