Skip to content
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

feat(2.0)!: remove mdxRs temporarily in loader and @rspress/modern-js-plugin #1851

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

SoonIter
Copy link
Member

@SoonIter SoonIter commented Feb 14, 2025

Summary

first breaking change

some milestones for 2.0

  1. remove mdxRs in mdx-loader
  2. add rehype-raw to mdx-loader
  3. migrate from prism to shiki
  4. pure esm infra and update shiki/mdxjs packages
  5. only extract toc and pageSearch with flattenMdx

Related Issue

Checklist

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

@SoonIter SoonIter changed the title feat(2.0)!: remove mdx in loader feat(2.0)!: remove mdxRs in loader Feb 14, 2025
@SoonIter SoonIter changed the base branch from syt/should-emit-error to main February 14, 2025 14:48
@SoonIter SoonIter force-pushed the syt/rspress/2.0-mdxRs branch from 62dc9b8 to 7f979be Compare February 27, 2025 03:39
Copy link

netlify bot commented Feb 27, 2025

Deploy Preview for aquamarine-blini-95325f ready!

Name Link
🔨 Latest commit 170a9d5
🔍 Latest deploy log https://app.netlify.com/sites/aquamarine-blini-95325f/deploys/67c01cf5bf1d330008d645e5
😎 Deploy Preview https://deploy-preview-1851--aquamarine-blini-95325f.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 62 (🔴 down 23 from production)
Accessibility: 97 (no change from production)
Best Practices: 92 (no change from production)
SEO: 100 (no change from production)
PWA: -
View the detailed breakdown and full score reports

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

@SoonIter SoonIter force-pushed the syt/rspress/2.0-mdxRs branch from 7f979be to 170a9d5 Compare February 27, 2025 08:06
@SoonIter SoonIter mentioned this pull request Feb 27, 2025
8 tasks
@SoonIter SoonIter changed the title feat(2.0)!: remove mdxRs in loader feat(2.0)!: remove mdxRs temporarily in loader Feb 27, 2025
@SoonIter SoonIter changed the title feat(2.0)!: remove mdxRs temporarily in loader feat(2.0)!: remove mdxRs temporarily in loader and @rspress/modern-js-plugin Feb 27, 2025
@SoonIter SoonIter marked this pull request as ready for review February 27, 2025 08:30
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.

1 participant