[llvm] [clang-doc] fix mustache template whitespace (PR #153724)
Erick Velez via llvm-commits
llvm-commits at lists.llvm.org
Sun Aug 24 14:33:44 PDT 2025
evelez7 wrote:
I'm not sure what you mean by other tools or how to solve this in JSON. My understanding is that the JSON just feeds the data to be rendered in the template, then this library handles the rendering, so the white space needs to be fixed here. AFAIK this library outputs the actual HTML file with all the entities written, so any whitespace issues come from here.
It'd be nice to have a test to check for exact whitespace offsets and indentation. I don't think whitespace matters in the lit suites but I believe it does in unit tests. Is indentation something that's tested in the Mustache spec tests @ilovepi ? We might want something like that if you can check indentation in the nodes before writing them to a file.
I'll try to checkout this patch and test manually to see if I see any of the alignment issues I mentioned, but it might not be until next week.
https://github.com/llvm/llvm-project/pull/153724
More information about the llvm-commits
mailing list