Skip to content

Fix for twigphp/Twig/issues/3685 (leading linebreaks and indentation)#6

Closed
Sarke wants to merge 1 commit intotwigphp:2.xfrom
Sarke:sarke-fix-Twig#3685
Closed

Fix for twigphp/Twig/issues/3685 (leading linebreaks and indentation)#6
Sarke wants to merge 1 commit intotwigphp:2.xfrom
Sarke:sarke-fix-Twig#3685

Conversation

@Sarke
Copy link
Copy Markdown

@Sarke Sarke commented Apr 15, 2022

Reworked the "remove indentation" to properly account for all lines, work with leading linebreaks, and retain codeblocks at start of markdown.

Reworked the tests, to among other things compare with the markdown generated by the markdown library directly.

@stof
Copy link
Copy Markdown
Member

stof commented Apr 16, 2022

This PR should be send to the Twig repo itself, as this repo is a read-only subtree split. See the extra folder in the Twig repo.

@fabpot fabpot closed this Apr 16, 2022
@Sarke
Copy link
Copy Markdown
Author

Sarke commented Apr 19, 2022

This PR should be send to the Twig repo itself, as this repo is a read-only subtree split. See the extra folder in the Twig repo.

Thanks, my bad.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants