nextcloud / helm

A community maintained helm chart for deploying Nextcloud on Kubernetes.
GNU Affero General Public License v3.0
295 stars 258 forks source link

Discussion: state of helm-docs for generating sectioned documentation #547

Open jessebot opened 3 months ago

jessebot commented 3 months ago

Per my comment to @wrenix here:

If we used helm-docs, that would be cool. It looks like they recently added the ability to do sections (which was largely what stopped us being interested in this before)! https://github.com/norwoodj/helm-docs/tree/master/example-charts/sections

We'd need to prefix each section with the name of the section like:

# @section -- database
postgresql:
# -- enable postgresql sub chart
  enabled: false

See their example values.yaml here: https://github.com/norwoodj/helm-docs/blob/master/example-charts/sections/values.yaml

But then you get a specific section you can render in the README.md like their example here:

## Values

### database

| Key | Type | Default | Description |
|-----|------|---------|-------------|
| postgresql.enabled | bool | `"false"` | enable postgresql sub chart |

Things to consider

open question

Can you have comments and additional data for specific sections? For instance, right now, we have a database section of our readme that gives further details on the externalDatabase parameter. Is there a way to call the sections in the README.gotmpl?

wrenix commented 2 months ago

I am fine with it, so every value option should be uncommented with an empty value to get it in the docs: https://github.com/nextcloud/helm/pull/534#discussion_r1485699750

(maybe you do not need any custom README.md.gotmpl just overwrite the default template for the chapter)


your question: i do not know it, but i believe a [database](#database) could work.