-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
refactor: add async loading for better performance #6643
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you make all markdown async and remove totally the sync version ?
Even better if we could remove markdownIt
, we have 2 markdown parser inside the application, we should keep only one
Co-authored-by: YannC <[email protected]>
Quality Gate passedIssues Measures |
Extract the Markdown parser to an asynchronous implementation, improving content parsing efficiency and enabling dynamic imports for better performance.
Make all routes loaded async
All the files
From globally -- 34.18MB of js
To globally ----- 28.97MB of js
Main file
From index.js at 16.28MB
To index at 6.28MB