Skip to content

chore: isolate docs from root workspace#941

Merged
fengmk2 merged 1 commit intomainfrom
chore/isolate-docs-workspace
Mar 16, 2026
Merged

chore: isolate docs from root workspace#941
fengmk2 merged 1 commit intomainfrom
chore/isolate-docs-workspace

Conversation

@branchseer
Copy link
Member

Remove docs from root workspace packages since it already has its own
pnpm-workspace.yaml for independent Netlify builds. Move vitepress
peer dep rule to docs workspace config. Add docs lockfile for
reproducible installs.

Co-Authored-By: Claude Opus 4.6 (1M context) noreply@anthropic.com

Copy link
Member Author

branchseer commented Mar 16, 2026

This stack of pull requests is managed by Graphite. Learn more about stacking.

@branchseer branchseer force-pushed the docs/add-mermaid-support branch from 207fbfd to a419862 Compare March 16, 2026 05:03
@branchseer branchseer force-pushed the chore/isolate-docs-workspace branch 2 times, most recently from d4aa513 to 99ba85d Compare March 16, 2026 05:10
@branchseer branchseer force-pushed the docs/add-mermaid-support branch from a419862 to bd2d465 Compare March 16, 2026 05:10
@branchseer branchseer force-pushed the chore/isolate-docs-workspace branch from 99ba85d to e04f516 Compare March 16, 2026 05:17
@branchseer branchseer force-pushed the docs/add-mermaid-support branch from bd2d465 to 60e43c6 Compare March 16, 2026 05:17
@branchseer branchseer force-pushed the chore/isolate-docs-workspace branch from e04f516 to c56f949 Compare March 16, 2026 05:22
@branchseer branchseer force-pushed the docs/add-mermaid-support branch from 60e43c6 to 6a90f89 Compare March 16, 2026 05:22
@branchseer branchseer force-pushed the chore/isolate-docs-workspace branch from c56f949 to ade1e3d Compare March 16, 2026 05:27
@branchseer branchseer force-pushed the docs/add-mermaid-support branch from 6a90f89 to 938c34f Compare March 16, 2026 05:27
@branchseer branchseer force-pushed the chore/isolate-docs-workspace branch 2 times, most recently from eee5a12 to 184f9b0 Compare March 16, 2026 06:31
Base automatically changed from docs/add-mermaid-support to main March 16, 2026 06:36
@branchseer branchseer force-pushed the chore/isolate-docs-workspace branch from 184f9b0 to c7bcbcf Compare March 16, 2026 06:45
@netlify
Copy link

netlify bot commented Mar 16, 2026

Deploy Preview for viteplus-preview ready!

Name Link
🔨 Latest commit a3b1eee
🔍 Latest deploy log https://app.netlify.com/projects/viteplus-preview/deploys/69b7c07f7e6f45000846a424
😎 Deploy Preview https://deploy-preview-941--viteplus-preview.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.

@cpojer
Copy link
Member

cpojer commented Mar 16, 2026

We should make sure that just init runs vp install for the docs folder then.

@branchseer branchseer force-pushed the chore/isolate-docs-workspace branch from c7bcbcf to bcc9514 Compare March 16, 2026 08:22
@branchseer
Copy link
Member Author

We should make sure that just init runs vp install for the docs folder then.

We don't require vp for vp developement yet. I've added pnpm install to the justfile in this PR, and leave the decision of using vp install to @fengmk2

@fengmk2
Copy link
Member

fengmk2 commented Mar 16, 2026

keep using pnpm install at vite-plus repo for now.

@branchseer branchseer marked this pull request as ready for review March 16, 2026 08:31
Remove docs from root workspace packages since it already has its own
pnpm-workspace.yaml for independent Netlify builds. Move vitepress
peer dep rule to docs workspace config. Add docs lockfile for
reproducible installs.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@branchseer branchseer force-pushed the chore/isolate-docs-workspace branch from bcc9514 to a3b1eee Compare March 16, 2026 08:34
@fengmk2 fengmk2 merged commit 8c6c1b2 into main Mar 16, 2026
22 checks passed
@fengmk2 fengmk2 deleted the chore/isolate-docs-workspace branch March 16, 2026 08:42
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.

3 participants