Repository navigation
docs(ecosystem): update package links and add nuxt.com - #480
Conversation
|
Note The title starts with
|
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Documentation previews📚 Preview all documentation changes (follows new pushes) Pinned to the current head: |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configuration
📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
Included review availability: This review used your included allowance. Your plan provides up to 4 included reviews per hour; 2 remain after this review. 📝 WalkthroughWalkthroughThe ecosystem catalog adds repository links and updates package entries. The showcase adds nuxt.com and includes it in the screenshot capture list. The Angular bundle-size test expectation changes to 56.4k. ChangesEcosystem documentation
Bundle-size snapshot
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~8 minutes Change: Other Merge Risk: ⚪ Minimal · up to This is a documentation update with a minor test expectation change, and no merge-blocking risk was found. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)✨ 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 |
comark
@comark/angular
@comark/ansi
@comark/html
@comark/nuxt
@comark/react
@comark/svelte
@comark/vue
commit: |
There was a problem hiding this comment.
Thanks for updating these links and adding the Nuxt showcase. I found a few items for maintainers:
comark-codemirroris inToolingbut has no requiredsubcategory: UI(line 91).- The chart entries still use deprecated
i-lucide-pie-chart; usei-lucide-chart-pieon lines 108 and 116. - This changes and removes entries attributed to
miguelrk, while the pull request author isfarnabaz, so a maintainer should confirm those updates.
The added WebP has the required 960×540 dimensions, and nuxt-com is included in the screenshot regeneration list.
|
Reviewed #480 and left one inline suggestion plus three notes for maintainers.
I left 1 suggestion in the review. The preview label was not enabled because the review also flags the missing Tooling subcategory, deprecated chart icons, and the ownership check for the |
Co-authored-by: comark-factory[bot] <335754239+comark-factory[bot]@users.noreply.github.com>
What
Point each
miguelrk/comark-packagesentry at its own directory in that monorepo, dropcomark-fetch(removed upstream), and renamecomark-flinttocomark-flint-chart. Add nuxt.com as the first card in Built with Comark.Why
Those packages all linked to the monorepo root, so every row opened the same page instead of the package itself.
comark-fetchno longer exists in the repo andcomark-flintwas renamed, so the old links would 404. nuxt.com renders its docs and pages through@comark/nuxtand belongs at the top of the showcase.Summary by CodeRabbit
comark-pdf,comark-email,comark-arrow,comark-codemirror, andcomark-etiketlistings.comark-flint-chartandcomark-vega, replaced thecomark-flintlisting withcomark-flint-chart, and removedcomark-fetch.@comark/angularto 56.4k; the expected file count remains unchanged.