-
Notifications
You must be signed in to change notification settings - Fork 971
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Match <!--more-->
in addition to <!-- more -->
#2397
Conversation
Can you add a test? |
Sorry for taking a bit to get to this-- a lot of my focus was on trying to get my particular static site setup working, and I was just submitting PRs for the changes I made that I thought would be reasonable to upstream. I added some tests directly to the markdown parser, since it seems that most of the current summary-splitting tests are in the content itself. This way, it's a little closer to a unit test than an integration test. Unfortunately, newline-split I also apologise that the test code is a little weird, adding in |
* Match <!-- more --> without spaces * Add tests for new <!-- more --> handling, with a note on pulldown-cmark bug
* Match <!-- more --> without spaces * Add tests for new <!-- more --> handling, with a note on pulldown-cmark bug
* Match <!-- more --> without spaces * Add tests for new <!-- more --> handling, with a note on pulldown-cmark bug
next
branch?I noticed this when trying to migrate a few WriteFreely posts to Zola. WriteFreely uses
<!--more-->
when added via the rich-text editor, whereas Zola was picky about the spaces around the tags. I replaced the check with a regex so that spaces and case don't matter any more.