Skip to content

feat: add Vue Plot - #1671

Open
memotux wants to merge 2 commits into
nuxt:mainfrom
memotux:vue-plot
Open

memotux wants to merge 2 commits into
nuxt:mainfrom
memotux:vue-plot

Conversation

@memotux

@memotux memotux commented Oct 13, 2026

Copy link
Copy Markdown

resolves #1591

Description

Add @memotux/nuxt-vue-plot to the modules listing.

Vue Plot wraps Observable Plot's grammar of graphics into idiomatic Vue components. The Nuxt module auto-imports the VPlot component, configures isCustomElement for Plot* tags, and optimizes Vite deps for @observablehq/plot.

@coderabbitai

coderabbitai Bot commented Oct 13, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: e56dea27-dd02-43f1-8b4a-ab084784373a
📥 Commits

Reviewing files that changed from the base of the PR and between 12b1c50 and acb767f.

⛔ Files ignored due to path filters: 1
  • icons/vue-plot.svg is excluded by !**/*.svg
📒 Files selected for processing: 1
  • modules/vue-plot.yml

Included review availability: This review used your included allowance. Your plan provides up to 4 included reviews per hour; 3 remain after this review.

📝 Walkthrough

Adds a vue-plot catalog entry. The entry includes Nuxt integration details, package and project links, an icon, category, third-party status, maintainer details, and a Nuxt compatibility requirement of >=4.4.7.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~3 minutes

Merge Risk: ⚪ Minimal · up to acb76

The Vue Plot catalog listing has no identified merge-blocking issue and is ready for normal checks.

Pre-merge checks: Passed 5
Passed Passed checks: 5 passed
Check name Status Explanation
Title check Passed The title clearly and concisely identifies the primary change: adding Vue Plot.
Description check Passed The description includes the linked issue, module purpose, package and repository details, category, compatibility, and documentation link. It satisfies the required template information.
Linked Issues check Passed Issue #1591 requests a Nuxt module listing for Vue Plot. modules/vue-plot.yml adds the @memotux/nuxt-vue-plot package, the memotux/vue-plot Nuxt repository, project and documentation links, a Li…
Out of Scope Changes check Passed The whole-PR summary shows one change in modules/vue-plot.yml. The change adds the requested Vue Plot catalog entry and its listing metadata. No unrelated implementation or documentation changes are…
Docstring Coverage Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Module Listing Request]: Vue Plot

1 participant