On Tue, 1 Oct 2024 at 15:52, Daniel Gustafsson <daniel@yesql.se> wrote:
> > So we need to think about a way to make this more robust for future people editing. Maybe something in
.gitattributesor some editor settings. Otherwise, it will be all over the places after a while.
>
> Maybe we can add some form of pandoc target for rendering as as way to test
> locally before pushing?
I think a gitattributes rule to disallow hard-tabs word work fine,
especially when combined with this patch of mine which keeps the
.editorconfig file in sync with the .gitattributes file:
https://commitfest.postgresql.org/49/4829/
> > Apart from this, I don't changing the placeholders like <foo> to < foo >. In some cases, this really decreases
readability. Maybe we should look for different approaches there.
>
> Agreed. I took a stab at some of them in the attached. The usage in
> src/test/isolation/README is seemingly the hardest to replace and I'm not sure
> how we should proceed there.
One way to improve the isolation/README situation is by:
1. indenting the standalone lines by four spaces to make it a code block
2. for the inline cases, replace <foo> with `<foo>` or `foo`