Skip to content

chore(deps): bump vite-plus to 0.1.21-alpha.7#3

Draft
fengmk2 wants to merge 3 commits into
mainfrom
update-vite-plus-alpha-0.1.21-alpha.7
Draft

chore(deps): bump vite-plus to 0.1.21-alpha.7#3
fengmk2 wants to merge 3 commits into
mainfrom
update-vite-plus-alpha-0.1.21-alpha.7

Conversation

@fengmk2
Copy link
Copy Markdown
Owner

@fengmk2 fengmk2 commented May 13, 2026

Summary

Bump vite-plus and related packages to alpha 0.1.21-alpha.7.

Updated where applicable:

  • vite-plus
  • vitest (alias to @voidzero-dev/vite-plus-test)
  • vite (alias to @voidzero-dev/vite-plus-core)
  • overrides / resolutions / pnpm.overrides / pnpm-workspace.yaml catalogs
  • minimumReleaseAgeExclude / minimumReleaseAgeExcludes configured for pnpm + bun

Test plan

  • CI passes

@fengmk2 fengmk2 self-assigned this May 13, 2026
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the versions for vite, vitest, and vite-plus in the pnpm workspace catalog from 'latest' to specific alpha versions. It also introduces a new configuration field, 'minimumReleaseAgeExclude', to exclude specific packages from release age checks. The reviewer suggested renaming this new field to 'minimumReleaseAgeExcludes' to ensure naming consistency with other pluralized list fields in the configuration file.

Comment thread pnpm-workspace.yaml
overrides:
vite: "catalog:"
vitest: "catalog:"
minimumReleaseAgeExclude:
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

The field name minimumReleaseAgeExclude is singular, which is inconsistent with other list-based fields in this file such as packages, overrides, and peerDependencyRules. Additionally, the pull request description mentions both minimumReleaseAgeExclude and minimumReleaseAgeExcludes, suggesting some uncertainty. For consistency with the existing file structure and to ensure the configuration is correctly recognized by the intended tools, consider using the plural form minimumReleaseAgeExcludes.

minimumReleaseAgeExcludes:

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.

1 participant