Skip to content

feat: docs for pluggable Markdown processors#13919

Merged
ArmandPhilippot merged 20 commits into
v6.4from
feat/markdown-agnostic
May 27, 2026
Merged

feat: docs for pluggable Markdown processors#13919
ArmandPhilippot merged 20 commits into
v6.4from
feat/markdown-agnostic

Conversation

@Princesseuh

@Princesseuh Princesseuh commented May 21, 2026

Copy link
Copy Markdown
Member

Description (required)

Docs for withastro/astro#16149 withastro/astro#16848

@netlify

netlify Bot commented May 21, 2026

Copy link
Copy Markdown

Deploy Preview for astro-docs-2 ready!

Name Link
🔨 Latest commit d71f973
🔍 Latest deploy log https://app.netlify.com/projects/astro-docs-2/deploys/6a16fbeed2d8cb0008894fc4
😎 Deploy Preview https://deploy-preview-13919--astro-docs-2.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.

@astrobot-houston

astrobot-houston commented May 21, 2026

Copy link
Copy Markdown
Contributor

Lunaria Status Overview

🌕 This pull request will trigger status changes.

Learn more

By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.

You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

File Note
en/guides/integrations-guide/mdx.mdx Source changed, localizations will be marked as outdated.
en/guides/markdown-content.mdx Source changed, localizations will be marked as outdated.
en/guides/migrate-to-astro/index.mdx Source changed, localizations will be marked as outdated.
en/reference/configuration-reference.mdx Source changed, localizations will be marked as outdated.
en/reference/content-loader-reference.mdx Source changed, localizations will be marked as outdated.
en/reference/modules/astro-content.mdx Source changed, localizations will be marked as outdated.
en/reference/programmatic-reference.mdx Source changed, localizations will be marked as outdated.
Warnings reference
Icon Description
🔄️ The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

@ArmandPhilippot ArmandPhilippot added merge-on-release Don't merge this before the feature is released! (MQ=approved but WAIT for feature release!) 7.0 labels May 21, 2026
@ArmandPhilippot ArmandPhilippot added this to the v7 milestone May 21, 2026

@ArmandPhilippot ArmandPhilippot left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work, Erika! I left a few comments, mostly nits and two questions where we now show how to do something with @astrojs/markdown-remark but without any equivalent for Sätteri.

And, as far as I can tell, the broken links should be fixed once your core PR is merged!

Comment thread src/content/docs/en/guides/integrations-guide/mdx.mdx Outdated
Comment thread src/content/docs/en/guides/markdown-content.mdx Outdated
Comment thread src/content/docs/en/guides/markdown-content.mdx Outdated
Comment thread src/content/docs/en/guides/markdown-content.mdx Outdated
Comment thread src/content/docs/en/guides/markdown-content.mdx Outdated
Comment thread src/content/docs/en/guides/markdown-content.mdx Outdated
Comment thread src/content/docs/en/guides/markdown-content.mdx Outdated
Comment thread src/content/docs/en/recipes/reading-time.mdx Outdated
Comment thread src/content/docs/en/guides/upgrade-to/v7.mdx Outdated
@Princesseuh Princesseuh removed the 7.0 label May 23, 2026
@Princesseuh Princesseuh modified the milestones: v7, 6.4 May 23, 2026
@Princesseuh Princesseuh changed the base branch from v7 to main May 23, 2026 02:03
@github-actions github-actions Bot added i18n Anything to do with internationalization & translation efforts - ask @YanThomas for help! 🚨 action labels May 23, 2026
Princesseuh and others added 5 commits May 23, 2026 04:07
Co-authored-by: Armand Philippot <git@armand.philippot.eu>
Co-authored-by: Armand Philippot <git@armand.philippot.eu>
Co-authored-by: Armand Philippot <git@armand.philippot.eu>
Co-authored-by: Armand Philippot <git@armand.philippot.eu>
@Princesseuh Princesseuh force-pushed the feat/markdown-agnostic branch from b13ef9c to 543fb80 Compare May 23, 2026 02:09
Comment thread src/content/docs/en/guides/integrations-guide/mdx.mdx
Comment thread src/content/docs/en/guides/migrate-to-astro/index.mdx Outdated
Comment thread src/content/docs/en/guides/markdown-content.mdx Outdated
Comment thread src/content/docs/en/guides/markdown-content.mdx Outdated

@ArmandPhilippot ArmandPhilippot left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the updates, Erika! I left a few suggestions (except the highlighting, this is mostly nitpicking). LGTM!

If you don't mind, I'll update the branch for #13943 and merge it in your PR. This way, we can fix link errors and ensure that each check is successful before the merge!

Comment thread src/content/docs/en/guides/migrate-to-astro/index.mdx Outdated
Comment thread src/content/docs/en/guides/markdown-content.mdx Outdated
Comment thread src/content/docs/en/guides/markdown-content.mdx Outdated
Comment thread src/content/docs/en/guides/markdown-content.mdx Outdated
Comment thread src/content/docs/en/guides/markdown-content.mdx
Co-authored-by: Armand Philippot <git@armand.philippot.eu>

@ArmandPhilippot ArmandPhilippot left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

GitHub wouldn't be GitHub without bugs... Again, instead of applying the suggestions it removed the content... hence the failing build.

Comment thread src/content/docs/en/guides/markdown-content.mdx Outdated
Comment thread src/content/docs/en/guides/markdown-content.mdx Outdated
Co-authored-by: Armand Philippot <git@armand.philippot.eu>
@ArmandPhilippot

Copy link
Copy Markdown
Member

What...

This commit does not include any file changes

Alright, I don't want to bother you with this again. I'll push the fix directly to your branch, merge main into it then the CI PR. Then, you can yell at me if I broke anything. 😄

@Princesseuh

Copy link
Copy Markdown
Member Author

ha, that's a new one. That's unfortunate

ArmandPhilippot and others added 3 commits May 27, 2026 16:06
Co-authored-by: delucis <357379+delucis@users.noreply.github.com>
Co-authored-by: Armand Philippot <git@armand.philippot.eu>
@ArmandPhilippot ArmandPhilippot added minor-release For the next minor release; in the milestone, "merge queue" = approved for merging on release day. and removed i18n Anything to do with internationalization & translation efforts - ask @YanThomas for help! 🚨 action labels May 27, 2026

@ArmandPhilippot ArmandPhilippot left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perfect, everything is green and I've double-checked I don't think GitHub broke more things!

@ArmandPhilippot

Copy link
Copy Markdown
Member

I'm merging this into an umbrella PR, so we'll only have one PR to worry about tomorrow.

@ArmandPhilippot ArmandPhilippot changed the base branch from main to v6.4 May 27, 2026 16:14
@ArmandPhilippot ArmandPhilippot merged commit 7602e89 into v6.4 May 27, 2026
12 checks passed
@ArmandPhilippot ArmandPhilippot deleted the feat/markdown-agnostic branch May 27, 2026 16:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merge-on-release Don't merge this before the feature is released! (MQ=approved but WAIT for feature release!) minor-release For the next minor release; in the milestone, "merge queue" = approved for merging on release day.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants