Skip to content

Markdown: Fix liquid flow parsing - #19838

Merged
fisker merged 5 commits into
prettier:mainfrom
seiyab:19730-review
Aug 15, 2026
Merged

fisker merged 5 commits into
prettier:mainfrom
seiyab:19730-review

Conversation

@fisker

@fisker fisker commented Aug 13, 2026 •

Copy link
Copy Markdown
Member

Description

Recreate #19730, since the author is not responding.

Fixes #19724

Checklist

  • I’ve added tests to confirm my change works.
  • (If changing the API or CLI) I’ve documented the changes I’ve made (in the docs/ directory).
  • (If the change is user-facing) I’ve added my changes to changelog_unreleased/*/XXXX.md file following changelog_unreleased/TEMPLATE.md.
  • I’ve read the contributing guidelines.
  • I did not use AI to generate this PR.
  • (If the above is not checked) I have reviewed the AI-generated content before submitting.

@netlify

netlify Bot commented Aug 13, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for prettier ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 1d0f196
🔍 Latest deploy log https://app.netlify.com/projects/prettier/deploys/6a7f94d56adeb60008a498a0
😎 Deploy Preview https://deploy-preview-19838--prettier.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@fisker fisker mentioned this pull request Aug 13, 2026
5 of 6 tasks
@pkg-pr-new

pkg-pr-new Bot commented Aug 13, 2026 •

Copy link
Copy Markdown

Open in StackBlitz

yarn add https://pkg.pr.new/@prettier/[email protected]
yarn add https://pkg.pr.new/@prettier/[email protected]
yarn add https://pkg.pr.new/@prettier/[email protected]
yarn add https://pkg.pr.new/[email protected]

commit: 1d0f196

@fisker
fisker marked this pull request as ready for review August 13, 2026 00:56
@fisker

fisker commented Aug 13, 2026 •

Copy link
Copy Markdown
Member Author

@seiyab I can't push to your branch, can you apply this?

diff --git "a/changelog_unreleased/markdown/19730.md" "b/changelog_unreleased/markdown/19730.md"
index 096f7a09c..82aa68636 100644
--- "a/changelog_unreleased/markdown/19730.md"
+++ "b/changelog_unreleased/markdown/19730.md"
@@ -1,6 +1,4 @@
-#### Preserve Liquid blocks after Markdown tables (#19730 by @wanxiankai)
-
-Standalone Liquid tags are now parsed as block syntax, preventing multiline tags from being absorbed into a preceding table or blockquote.
+#### Preserve Liquid blocks after Markdown tables (#19730 by @wanxiankai, @seiyab)
 
 <!-- prettier-ignore -->
 ```markdown

@fisker fisker changed the title Fix Markdown Liquid flow parsing Markdown: Fix liquid flow parsing Aug 15, 2026
@fisker
fisker merged commit 560146e into prettier:main Aug 15, 2026
36 checks passed
@fisker fisker mentioned this pull request Sep 10, 2026
2 of 6 tasks
fisker added a commit that referenced this pull request Sep 16, 2026
Co-authored-by: codex <[email protected]>
Co-authored-by: fisker Cheung <[email protected]>
Co-authored-by: wanxiankai <[email protected]>
Co-authored-by: seiya <[email protected]>
@fisker fisker mentioned this pull request Sep 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[3.9 regression] Markdown: multi-line {% … %} statement after a table is absorbed into the table, corrupting Jinja/Liquid templates

3 participants