Michael Osipov

I don't understand this change. It does not retain the original format, what does it address?

> It fixes the indentation behaviour when the source contains tabs. Without the fix the indentation is not correctly handled. Without the fix the source code block has too many...

First snippet has two tabs, but expected does not have any. What do I miss?

> The tabs from this example `[tab][tab]line` will not be removed. Thus the snippet looks ugly. The tabs should be removed. Why? If the snippet is intended that way? How...

@kwin, do you understand this PR?

But what if ``` { // your code } ``` is correct indentation?

> The code clearly removes this indentation if it is done by using spaces. Using tabs it is not removed and thus looks broken. I'm okay with it if that...

> I think the indentation is clearly removed in order to prevent horizontal scrollbars for included snippets. I'd prefer to leave formatting as-is, but I am open to a snippet...

I'd like to perform a release this weekend. Anyone want to complete the discussion with me?

> And should we also change in: > > https://github.com/apache/maven-parent/blob/c6750dfc7a8aa7161d0afd56550ab79965b331e9/docs/src/site/site.xml#L26-L30 Good catch!