Skip to content

Conversation

@emmanuelmathot
Copy link
Contributor

@emmanuelmathot emmanuelmathot commented Apr 20, 2025

Description

This PR resolves Issue #16 by implementing Artifacthub.io integration for better Helm chart discoverability at https://artifacthub.io/packages/search?repo=eoapi&sort=relevance&page=1

Changes

  • Added Artifacthub.io specific annotations to both eoapi and postgrescluster Chart.yaml files
  • Added keywords, maintainers, links, and changes metadata for better chart discoverability
  • Removed invalid force property from postgrescluster Chart.yaml
  • Updated the README with an Artifacthub.io repository badge and link
  • Cleaned up the release workflow file by removing trailing whitespace

Related Issues

Closes #16

Testing

The changes add metadata to the Helm charts for Artifacthub.io integration. The updates to the Chart.yaml files are validated to ensure they follow the proper schema.

@emmanuelmathot
Copy link
Contributor Author

official status request: artifacthub/hub#4358

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR implements Artifacthub.io integration for the Helm charts to improve discoverability. Key changes include:

  • Adding Artifacthub.io metadata (annotations for changes, links, maintainers, and keywords) to both the postgrescluster and eoapi Chart.yaml files
  • Removing an invalid force property from the postgrescluster Chart.yaml
  • Updating the README with an Artifacthub.io repository badge and cleaning up the release workflow file

Reviewed Changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated no comments.

File Description
helm-chart/postgrescluster/Chart.yaml Removed the invalid force property and added Artifacthub.io metadata
helm-chart/eoapi/Chart.yaml Added Artifacthub.io metadata
README.md Updated badge information and added an Artifacthub.io badge
.github/workflows/release.yml Updated the condition expression and removed trailing whitespace
Files not reviewed (1)
  • artifacthub-repo.yml: Language not supported

Copy link
Contributor

@pantierra pantierra left a comment

Choose a reason for hiding this comment

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

LGTM

@emmanuelmathot emmanuelmathot merged commit 9e6e72b into main Apr 28, 2025
2 checks passed
@emmanuelmathot emmanuelmathot deleted the artifacthub branch April 28, 2025 15:25
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.

CI/CD Artifacthub.io Integration

3 participants