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

Update artifact name #24

Merged
merged 1 commit into from
Jan 13, 2021

Conversation

VijayanB
Copy link
Member

prefix artifact name with opendistro-.

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

prefix artifact name with opendistro-.
@VijayanB VijayanB self-assigned this Jan 13, 2021
@codecov
Copy link

codecov bot commented Jan 13, 2021

Codecov Report

Merging #24 (c68e6a3) into main (1bd9b6f) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main      #24   +/-   ##
=======================================
  Coverage   65.71%   65.71%           
=======================================
  Files          16       16           
  Lines        1120     1120           
=======================================
  Hits          736      736           
  Misses        304      304           
  Partials       80       80           
Flag Coverage Δ
odfe-cli 65.71% <ø> (ø)

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

Copy link
Member

@vamshin vamshin left a comment

Choose a reason for hiding this comment

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

LGTM!

@VijayanB VijayanB merged commit b38b67f into opendistro-for-elasticsearch:main Jan 13, 2021
@VijayanB VijayanB deleted the fix-name branch January 13, 2021 20:59
@VijayanB VijayanB added the Infrastructure A change to infrastructure, testing, CI/CD, pipelines, etc. label Feb 4, 2021
VijayanB added a commit to VijayanB/es-cli that referenced this pull request Feb 10, 2021
prefix artifact name with opendistro-*.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Infrastructure A change to infrastructure, testing, CI/CD, pipelines, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants