Skip to content

chore(template): Use .yaml extension for artifacthub-repo.yml - #638

Merged
Techassi merged 1 commit into
mainfrom
chore/template-rename-artifacthub-repo-file
Aug 17, 2026
Merged

chore(template): Use .yaml extension for artifacthub-repo.yml#638
Techassi merged 1 commit into
mainfrom
chore/template-rename-artifacthub-repo-file

Conversation

@Techassi

Copy link
Copy Markdown
Member

We generelly discourage using .yml for YAML files. Artifact Hub supports both according to their tests and code.

This rename also resulted in yamllint to pick up the file.

We generelly discourage using .yml for YAML files. Artifact Hub
supports both according to their tests and code.
@Techassi Techassi self-assigned this Aug 17, 2026
@Techassi Techassi moved this to Development: Waiting for Review in Stackable Engineering Aug 17, 2026
@lfrancke

lfrancke commented Aug 17, 2026

Copy link
Copy Markdown
Member

That's the file name required by ArtifactHub: https://artifacthub.io/docs/topics/repositories/#verified-publisher

I have not checked whether it also accepts .yaml

Edit: If it does I'm absolutely fine to rename this.

@Techassi

Copy link
Copy Markdown
Member Author

I have not checked whether it also accepts .yaml

Yup, it does. Both via OCI and HTTP(S).

https://github.com/artifacthub/hub/blob/f90cbb4ee79a31eaa348dbb2868ffa9d2cbe15d8/internal/repo/manager.go#L417-L440

@NickLarsenNZ NickLarsenNZ moved this from Development: Waiting for Review to Development: In Review in Stackable Engineering Aug 17, 2026
@Techassi
Techassi added this pull request to the merge queue Aug 17, 2026
@Techassi Techassi moved this from Development: In Review to Development: Done in Stackable Engineering Aug 17, 2026
Merged via the queue into main with commit 4aacf58 Aug 17, 2026
2 checks passed
@Techassi
Techassi deleted the chore/template-rename-artifacthub-repo-file branch August 17, 2026 11:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Development: Done

Development

Successfully merging this pull request may close these issues.

3 participants