Skip to content

docs(migration): add mdxRs option removal to migration guide#3196

Merged
SoonIter merged 1 commit intoweb-infra-dev:mainfrom
tats-u:migration-guide-mdxrs
Mar 4, 2026
Merged

docs(migration): add mdxRs option removal to migration guide#3196
SoonIter merged 1 commit intoweb-infra-dev:mainfrom
tats-u:migration-guide-mdxrs

Conversation

@tats-u
Copy link
Copy Markdown
Contributor

@tats-u tats-u commented Mar 4, 2026

Summary

The Rspress 2.0 migration guide omits the removal of markdown.mdxRs, though the blog announcement covers it. Users keeping this option hit TypeScript type errors with no explanation in the migration docs.

Notice: Copilot powered by Claude Opus 4.6 was used

Related Issue

Followup of the PRs #2940, #2954, #3075

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: Tatsunori Uchino <tats.u@live.jp>
Copilot AI review requested due to automatic review settings March 4, 2026 15:07
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds missing migration-guide documentation for the removal of the markdown.mdxRs config option in Rspress v2, aligning the migration docs with the v2 announcement blog so users understand the resulting TypeScript config/type errors.

Changes:

  • Add markdown.mdxRs removal to the quick migration checklist (EN/ZH).
  • Add a dedicated migration section explaining that markdown.mdxRs is removed and should be deleted from config, with before/after examples (EN/ZH).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
website/docs/zh/guide/migration/rspress-1-x.mdx Documents markdown.mdxRs removal in the Chinese migration guide, including checklist + before/after config example.
website/docs/en/guide/migration/rspress-1-x.mdx Documents markdown.mdxRs removal in the English migration guide, including checklist + before/after config example.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@SoonIter SoonIter enabled auto-merge (squash) March 4, 2026 16:15
@SoonIter SoonIter merged commit 3ba4c40 into web-infra-dev:main Mar 4, 2026
9 of 10 checks passed
@SoonIter
Copy link
Copy Markdown
Member

SoonIter commented Mar 4, 2026

thanks ❤~

@tats-u tats-u deleted the migration-guide-mdxrs branch March 4, 2026 21:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants