-
Notifications
You must be signed in to change notification settings - Fork 56
Update artifact naming #309
Update artifact naming #309
Conversation
Codecov Report
@@ Coverage Diff @@
## master #309 +/- ##
=========================================
Coverage 80.51% 80.51%
Complexity 388 388
=========================================
Files 62 62
Lines 1468 1468
Branches 130 130
=========================================
Hits 1182 1182
Misses 239 239
Partials 47 47 Continue to review full report at Codecov.
|
@jmazanec15 Will the updated name appears in github checks now? |
Yes workflows will still work. We use wildcards for picking up names. For example, https://github.com/opendistro-for-elasticsearch/k-NN/blob/master/.github/workflows/CD.yml#L58-L60 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Issue #, if available:
Description of changes:
This PR updates the k-NN plugins artifacts with the naming conventions set by the infrastructure team.
The new artifacts look like this:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.