-
-
Notifications
You must be signed in to change notification settings - Fork 11
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
Add support for multiline comments above property #52
Comments
Top comment is a bit of a problem because of |
And what about being compatible with |
I was referring more to development of this plugin. When do you start reading comments and when do you stop? Is there something before or after in comments that should be ignored? Is it with annotation for schema, helm-docs or regular user comment? I avoided all of that by reading line comment. I understand that having all those things is very convenient for end users but moving to top comment will also introduce breaking change for existing users. |
My 2 cents:
Side node: I'm loving this tool, thanks a bunch @losisin for putting it out there, it's been perfect for my use case. If you turn on your sponsor button I want to buy you a beer or two |
Different idea: if the image:
repository: registry.example.com/foo
tag: 1.2.3
# @schema ref: image; description: Image settings
# @schema ref: image.repository; description: The repository to use; required: true; default: registry.example.com/foo
# @schema ref: image.tag; description: The tag to use A side benefit would be I can do things this way and make the values.yaml file more readable for downstream users. It takes a lot of brain power to grok the yaml among all the line comments. This example doesn't demonstrate the original ask of this issue though, as it still does just one line for each. Not sure how multiline might look, maybe something like
IDK, I'm sure there's probably a better way out there. |
I like the idea of adding a ref and the |
Regarding #51 it would also be very helpful to allow comments above a property to be parsed for the schema tags.
For example:
The above also defaults a non annotated comment above a property to use as the
description
. Maybe this could be turned on with a flag in the cli?The text was updated successfully, but these errors were encountered: