Skip to content

feat: added toHTML function for markdown to HTML conversion#262

Closed
AndrewChubatiuk wants to merge 10 commits into
norwoodj:masterfrom
AndrewChubatiuk:tohtml-func
Closed

feat: added toHTML function for markdown to HTML conversion#262
AndrewChubatiuk wants to merge 10 commits into
norwoodj:masterfrom
AndrewChubatiuk:tohtml-func

Conversation

@AndrewChubatiuk

@AndrewChubatiuk AndrewChubatiuk commented Aug 19, 2024

Copy link
Copy Markdown
  • using chart.valuesSectionHtml template requires to convert all markdown syntax in values.yaml comments to HTML, which makes it less readable. Added toHTML function to perform this conversion
  • moved templates to file for convenience, which is included during build time using embed package
  • added include template function like in Helm to assign template output to variable (fixes Implement something like helm's include function #170)
  • added highlight function to convert codeblock to HTML and to highlight it using chroma
  • merged "Key", "Default" into "Key" and "Type", "Description" into "Description" columns to make table more compact

@AndrewChubatiuk AndrewChubatiuk changed the title feat: added toHTML function for markdown to HTML convertsion feat: added toHTML function for markdown to HTML conversion Oct 4, 2024
@AndrewChubatiuk
AndrewChubatiuk force-pushed the tohtml-func branch 10 times, most recently from e240883 to 342ad8d Compare February 25, 2025 12:26
@AndrewChubatiuk
AndrewChubatiuk force-pushed the tohtml-func branch 18 times, most recently from 63f9852 to 2e1e62c Compare April 13, 2025 04:51
@AndrewChubatiuk
AndrewChubatiuk force-pushed the tohtml-func branch 7 times, most recently from 4795227 to c322e52 Compare April 13, 2025 06:58
@norwoodj

norwoodj commented Mar 4, 2026

Copy link
Copy Markdown
Owner

Gotta resolve your merge conflicts

@AndrewChubatiuk
AndrewChubatiuk force-pushed the tohtml-func branch 3 times, most recently from 143551d to 294c326 Compare March 4, 2026 21:26
@AndrewChubatiuk

Copy link
Copy Markdown
Author

divided this PR into #338, #339, #340

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.

Implement something like helm's include function

2 participants