Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

CLI: Fix build errors #235

Merged
merged 2 commits into from
Sep 24, 2020
Merged

Conversation

VijayanB
Copy link
Member

Added fmt to headers

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Added fmt to headers
@codecov
Copy link

codecov bot commented Sep 23, 2020

Codecov Report

Merging #235 into master will decrease coverage by 2.19%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #235      +/-   ##
==========================================
- Coverage   80.30%   78.11%   -2.20%     
==========================================
  Files           6        6              
  Lines         716      626      -90     
==========================================
- Hits          575      489      -86     
+ Misses         84       80       -4     
  Partials       57       57              
Flag Coverage Δ
#cli 78.11% <ø> (-2.20%) ⬇️
#plugin 100.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
cli/internal/gateway/ad/ad.go 60.43% <0.00%> (-4.95%) ⬇️
cli/internal/controller/ad/ad.go 75.17% <0.00%> (-2.22%) ⬇️
cli/internal/gateway/es/es.go 84.61% <0.00%> (-2.06%) ⬇️
cli/internal/mapper/ad/ad.go 85.24% <0.00%> (-1.19%) ⬇️
cli/internal/controller/es/es.go 100.00% <0.00%> (ø)
cli/internal/handler/ad/ad.go 90.10% <0.00%> (+0.01%) ⬆️

@VijayanB VijayanB self-assigned this Sep 23, 2020
Allow flexible threshold for code coverage.
@VijayanB VijayanB merged commit 0405895 into opendistro-for-elasticsearch:master Sep 24, 2020
@ohltyler ohltyler added the enhancement New feature or request label Oct 19, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants