-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Drop cheerio #1299
Drop cheerio #1299
Conversation
Here is an overview of what got changed by this pull request: Issues
======
+ Solved 2
See the complete overview on Codacy |
We never have any test case but seems everything's ok 😘 |
I wonder why replace cheerio with regex..? |
"dependencies": {
- "cheerio": "0.22.0",
},
"devDependencies": {
+ "cheerio": "0.22.0",
} |
But... use regex to handle html? Why not just a use another html parser or similar tools such as jsdom or other lightweight alternative? |
Please note: Currently the theme is installed via |
另一个方案是,重写md渲染器 比如:jiangtj/hexo-theme-cake#28 提供了 |
维护一个主题已经很累了。渲染器不关我的事,谁爱管谁管。 |
😂 这几天在研究这个,hexo如果只用一个固定的md渲染器,整体性能应该能提高至少50% |
确实如此,而且像 hexo-renderer-kramed 这种已经几年没更新了 https://www.npmjs.com/package/hexo-renderer-kramed 有兴趣的话可以发邮件给npm接手这个包。
|
No description provided.