Skip to content

chore: add CONTRIBUTING.md and fix monorepo paths in generate-api.js#558

Merged
gka merged 1 commit into
svelteplot:mainfrom
RobertFrenken:chore/contributing
Apr 27, 2026
Merged

chore: add CONTRIBUTING.md and fix monorepo paths in generate-api.js#558
gka merged 1 commit into
svelteplot:mainfrom
RobertFrenken:chore/contributing

Conversation

@RobertFrenken

Copy link
Copy Markdown
Contributor

Adds a CONTRIBUTING.md at the repo root following the convention used by Vega-Lite, Svelte, and shadcn/ui — single root file covering project layout, adding a new mark, adding a new transform, and per-PR expectations.

Also fixes scripts/generate-api.js, which had stale src/lib/... paths from before the monorepo refactor (#551). pnpm docs:api:marks (referenced from CONTRIBUTING.md) won't work without this fix.

Both pieces were originally bundled with #552 but they're project-wide tooling, not part of the hexbin feature, so splitting them out so they can land independently.

🤖 Generated with Claude Code

CONTRIBUTING.md follows the convention used by Vega-Lite, Svelte, and
shadcn/ui — single root file with a section on adding a new mark
absorbing what was previously a separate NEW_MARK_CHECKLIST.md.

scripts/generate-api.js had stale src/lib/... paths from before the
monorepo refactor (3a1d520); updated to packages/svelteplot/src/...
so pnpm docs:api:marks (referenced from CONTRIBUTING.md) works again.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@netlify

netlify Bot commented Apr 27, 2026

Copy link
Copy Markdown

Deploy Preview for svelteplot ready!

Name Link
🔨 Latest commit 0829a40
🔍 Latest deploy log https://app.netlify.com/projects/svelteplot/deploys/69eeabc5f130de000862ec55
😎 Deploy Preview https://deploy-preview-558--svelteplot.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@gka gka merged commit b32ba40 into svelteplot:main Apr 27, 2026
9 checks passed
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.

2 participants