Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: add artifact hub metadata #372

Merged
merged 10 commits into from
Mar 16, 2023
Merged

chore: add artifact hub metadata #372

merged 10 commits into from
Mar 16, 2023

Conversation

beeme1mr
Copy link
Member

@beeme1mr beeme1mr commented Mar 7, 2023

This PR

  • adds additional ArtifactHUB metadata in the Chart.yaml

Notes

The documentation for the metadata accepted by ArtifactHUB can be found here.

Follow-up Tasks

Double-check the ArtifactHUB entry after this is merged and OFO was been updated.

How to test

This hasn't been tested but the changes match the ArtifcatHUB documentation. It's low risk unless the yaml is invalid.

https://artifacthub.io/docs/topics/annotations/helm/

Signed-off-by: Michael Beemer <beeme1mr@users.noreply.github.com>
Signed-off-by: Michael Beemer <beeme1mr@users.noreply.github.com>
@codecov
Copy link

codecov bot commented Mar 7, 2023

Codecov Report

Merging #372 (f9e6f21) into main (41efb15) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #372   +/-   ##
=======================================
  Coverage   71.29%   71.29%           
=======================================
  Files          11       11           
  Lines         836      836           
=======================================
  Hits          596      596           
  Misses        213      213           
  Partials       27       27           
Flag Coverage Δ
component-tests 64.91% <ø> (ø)
unit-tests 21.05% <ø> (ø)

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

Signed-off-by: Michael Beemer <beeme1mr@users.noreply.github.com>
Signed-off-by: Michael Beemer <beeme1mr@users.noreply.github.com>
https://artifacthub.io/docs/topics/annotations/helm/#supported-annotations

Alternative names must be a substring of the name. Unfortunately, that means ofo is invalid.

Signed-off-by: Michael Beemer <beeme1mr@users.noreply.github.com>
Signed-off-by: Michael Beemer <beeme1mr@users.noreply.github.com>
Signed-off-by: Michael Beemer <beeme1mr@users.noreply.github.com>
Signed-off-by: Michael Beemer <beeme1mr@users.noreply.github.com>
Signed-off-by: Michael Beemer <beeme1mr@users.noreply.github.com>
@beeme1mr beeme1mr merged commit c6f539f into main Mar 16, 2023
@beeme1mr beeme1mr deleted the beeme1mr-patch-1 branch March 16, 2023 19:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants