fix: add fragments to headings#1807
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
1 Skipped Deployment
|
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review infoConfiguration used: Organization UI Review profile: CHILL Plan: Pro ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (2)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughWalkthroughImports and registers the markdown-it-anchor plugin in the blog module's Markdown-it setup (calls md.use(MarkdownItAnchor)) and adds the "markdown-it-anchor" dependency to package.json. No other control-flow or public API changes were made. Suggested reviewers
🚥 Pre-merge checks | ✅ 1✅ Passed checks (1 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
“Validate PR title” failure addressed in #1808. Changing the PR title and commit message to one that will pass the tests, to make this ready for review. |
df01762 to
2e40fdd
Compare
🔗 Linked issue
This fix was trivial so I decided to just create a PR.
🧭 Context
On the blog, headings do not have fragments, so cannot be linked to:
📚 Description
I used markdown-it-anchor to automatically add fragments to headings: