docs: add monorepo overrides guide#1535
Conversation
✅ Deploy Preview for viteplus-preview ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
How to use the Graphite Merge QueueAdd the label auto-merge to this PR to add it to the merge queue. You must have a Graphite account in order to use the merge queue. Sign up using this link. An organization admin has enabled the Graphite Merge Queue in this repository. Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue. This stack of pull requests is managed by Graphite. Learn more about stacking. |
51fd085 to
d7e039a
Compare
fdada6f to
4124ede
Compare
|
@cursor review |
There was a problem hiding this comment.
✅ Bugbot reviewed your changes and found no new issues!
Comment @cursor review or bugbot run to trigger another review on this PR
Reviewed by Cursor Bugbot for commit 7597248. Configure here.
leaysgur
left a comment
There was a problem hiding this comment.
I think the document itself looks fine.
I would like to confirm my understanding: we are going to continue always appending -c for vp lint|fmt, right?
In that case:
- #1378 and #1380 will become unnecessary
- Since we cannot force the equivalent of
-con the LSP side, we need to preventlint/fmtfrom being written in a non-rootvite.config.ts- We will cover this with lint rules and avoid modifying the oxapps implementation
Is this correct?
There's no final conclusion yet, but we tend to configure lint and fmt only in the root config and see if this recommended solution can meet the vast majority of needs. Yes, after this document PR is merged, I will close these 2 PRs.
Yes, after our oxlint plugin is merged, new rules will be added to intercept it. |
Co-authored-by: Christoph Nakazawa <xnakazawa@pm.me> Signed-off-by: MK (fengmk2) <fengmk2@gmail.com>
9763ae8 to
0687466
Compare

Note
Medium Risk
Low product risk (docs-only content), but CI risk is moderate because the E2E matrix adds a new external fixture that can introduce flaky failures or longer runtimes.
Overview
Adds a new Monorepo guide documenting how to use root
vite.config.tswithlint.overridesandfmt.overridesfor package-specific rules/options, including examples for splitting override configs into separate files.Wires the new guide into the docs sidebar and updates the
lint/fmtconfig docs to point monorepo users at the override approach.Extends ecosystem CI by adding the
vite-plus-monorepo-overridesfixture torepo.jsonand running it in the E2E workflow (vp check --fix+vp run verify).Reviewed by Cursor Bugbot for commit 7597248. Configure here.