Conversation
liangmiQwQ
left a comment
There was a problem hiding this comment.
Hi! Please update this snapshot to fix CI failing https://github.com/voidzero-dev/vite-plus/blob/2d2548275120cc6f66054dca23b1dba39c494793/crates/vp_cli_snapshots/tests/cli_snapshots/fixtures/global_cli_fallback/snapshots/global_cli_fallback.md
|
@liangmiQwQ I'm curious why this skill at https://github.com/voidzero-dev/vite-plus/blob/main/.claude/skills/sync-upstream-cli-help/SKILL.md didn't take effect. |
@fengmk2 I think the reason is that we manually update Vitest major in #2551, which won't trigger the skill. |
liangmiQwQ
left a comment
There was a problem hiding this comment.
LGTM if CI is green.
vp test --helpomits five top-level options accepted by the bundled Vitest 5, so users cannot discover them through the CLI. The static help list also still refers toexperimental.fsModuleCacheafter that option moved to the top level.Add
--repeats,--injectCjsGlobals,--fsModuleCache,--fsModuleCachePath, and--sharedViteServerwith their upstream descriptions and defaults, and correct the--clearCachedescription. Update the recorded output for both help aliases. The change is limited to help text and its snapshots.Fixes #2804.