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

Install go get outside cli directory #216

Merged
merged 1 commit into from
Aug 25, 2020

Conversation

VijayanB
Copy link
Member

go get inside cli will update the go.mod, since this is not part of code dependencies,
install goimports outside scope of cli.

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

@VijayanB VijayanB self-assigned this Aug 25, 2020
@VijayanB VijayanB added the AnomalyDetection Items related to Anomaly Detection plugin label Aug 25, 2020
@VijayanB VijayanB requested review from wnbts, kaituo and ylwu-amzn August 25, 2020 00:06
@codecov
Copy link

codecov bot commented Aug 25, 2020

Codecov Report

Merging #216 into master will increase coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #216      +/-   ##
============================================
+ Coverage     72.40%   72.41%   +0.01%     
- Complexity     1289     1290       +1     
============================================
  Files           139      139              
  Lines          6073     6073              
  Branches        469      469              
============================================
+ Hits           4397     4398       +1     
  Misses         1464     1464              
+ Partials        212      211       -1     
Flag Coverage Δ Complexity Δ
#cli 80.30% <ø> (ø) 0.00 <ø> (ø)
#plugin 71.36% <ø> (+0.01%) 1290.00 <ø> (+1.00)

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

Impacted Files Coverage Δ Complexity Δ
cli/internal/controller/ad/ad.go 77.38% <0.00%> (ø) 0.00% <0.00%> (ø%)
...sticsearch/ad/indices/AnomalyDetectionIndices.java 62.31% <0.00%> (+0.72%) 24.00% <0.00%> (+1.00%)

@VijayanB VijayanB merged commit 9df7b59 into opendistro-for-elasticsearch:master Aug 25, 2020
@VijayanB VijayanB added infra and removed AnomalyDetection Items related to Anomaly Detection plugin labels Aug 25, 2020
yizheliu-amazon pushed a commit to yizheliu-amazon/anomaly-detection that referenced this pull request Aug 28, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants