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

CLI: Update Display Strings #231

Merged
merged 1 commit into from
Sep 23, 2020

Conversation

VijayanB
Copy link
Member

To make it consistent, short message of commands first character
will be an uppercase ,flags first character will be lowercase.
Include Description as Header for command's long message.

Screen Shot 2020-09-14 at 12 53 47 PM

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

To make it consistent, short message of commands first character
will be an uppercase ,flags will first character will be lowercase.
Include Description as Header for command's long message.
@VijayanB VijayanB added documentation Improvements or additions to documentation enhancement New feature or request labels Sep 14, 2020
@VijayanB VijayanB self-assigned this Sep 14, 2020
@codecov
Copy link

codecov bot commented Sep 14, 2020

Codecov Report

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

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #231      +/-   ##
============================================
+ 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 Δ
...sticsearch/ad/indices/AnomalyDetectionIndices.java 62.31% <0.00%> (+0.72%) 24.00% <0.00%> (+1.00%)

@VijayanB VijayanB requested a review from wnbts September 17, 2020 02:26
@VijayanB VijayanB merged commit 890b9df into opendistro-for-elasticsearch:master Sep 23, 2020
@VijayanB VijayanB deleted the ImproveUX branch September 23, 2020 18:35
VijayanB added a commit that referenced this pull request Sep 23, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants