Skip to content

Add HULL - Helm Uniform Layer Library - Schema - #6292

Merged
madskristensen merged 5 commits into
SchemaStore:masterfrom
gre9ory:master
Sep 3, 2026
Merged

Add HULL - Helm Uniform Layer Library - Schema#6292
madskristensen merged 5 commits into
SchemaStore:masterfrom
gre9ory:master

Conversation

@gre9ory

@gre9ory gre9ory commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

No description provided.

Add HULL schema
remove file patterns for HULL schema
@github-actions

github-actions Bot commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Thanks for the PR!

This section of the codebase is owned by @madskristensen and @hyperupcall - if they write a comment saying "LGTM" then it will be merged.

@madskristensen

Copy link
Copy Markdown
Contributor

The fileMatch *values*.yaml is too generic. It would apply this schema to ordinary Helm values files, not just HULL charts. Please tighten it to a filename or path that uniquely identifies HULL config.

@gre9ory

gre9ory commented Sep 3, 2026

Copy link
Copy Markdown
Contributor Author

@madskristensen Thank you for the response.

I don't think there is any agreed upon filenaming scheme for additional overlay files to Helm charts.

Within the chart the values.yaml is a defined name but any additional file that is merged with this via the -f <filename> argument can have any arbitrary name.

Would it be acceptable to not have a fileMatch in this case? I see there are many other examples that omit fileMatch in the specification such as:

    {
      "name": "Ethereum ERC721",
      "description": "ERC-721 Non-Fungible Token Standard",
      "url": "https://www.schemastore.org/ethereum-erc721.json"
    },

    {
      "name": "mod.pw.toml (packwiz)",
      "description": "A metadata file which references an external file from a URL",
      "url": "https://www.schemastore.org/packwiz-mod.json"
    },
    {
      "name": "pack.toml (packwiz)",
      "description": "The main modpack file for a packwiz modpack",
      "url": "https://www.schemastore.org/packwiz-pack.json"
    },

@madskristensen

Copy link
Copy Markdown
Contributor

Yes, fileMatch is an optional property

@gre9ory

gre9ory commented Sep 3, 2026

Copy link
Copy Markdown
Contributor Author

Alright, already removed it.

@madskristensen
madskristensen merged commit 594d243 into SchemaStore:master Sep 3, 2026
4 checks passed
@madskristensen

Copy link
Copy Markdown
Contributor

Thanks

@gre9ory

gre9ory commented Sep 3, 2026

Copy link
Copy Markdown
Contributor Author

Appreciated!

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.

2 participants