Skip to content

Conversation

@chenjiahan
Copy link
Member

No description provided.

Copilot AI review requested due to automatic review settings January 15, 2026 03:54
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request updates the peerDependencies specification for @rsbuild/core to support Rsbuild v2 by adding ^2.0.0-0 to the version range. The change allows the plugin to work with both Rsbuild v1.7.0 and above, as well as all v2 pre-release and stable versions.

Changes:

  • Updated @rsbuild/core peerDependencies version range to support both v1.7.0+ and v2.0.0+

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

},
"peerDependencies": {
"@rsbuild/core": "^1.7.0-beta.0 || ^1.7.0"
"@rsbuild/core": "^1.7.0 || ^2.0.0-0"
Copy link

Copilot AI Jan 15, 2026

Choose a reason for hiding this comment

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

The README.md still only references @rsbuild/core >= 1.7.0 in the installation instructions (line 25). The documentation should be updated to mention that this version also supports Rsbuild v2, or the installation instructions should specify the version range more clearly to reflect the new peerDependencies.

Suggested change
"@rsbuild/core": "^1.7.0 || ^2.0.0-0"
"@rsbuild/core": ">=1.7.0"

Copilot uses AI. Check for mistakes.
},
"peerDependencies": {
"@rsbuild/core": "^1.7.0-beta.0 || ^1.7.0"
"@rsbuild/core": "^1.7.0 || ^2.0.0-0"
Copy link

Copilot AI Jan 15, 2026

Choose a reason for hiding this comment

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

There are no tests verifying compatibility with Rsbuild v2. The existing test in test/basic/index.test.ts uses a devDependency on v1.7.0-beta.0. Consider adding tests or updating the devDependency to verify the plugin works correctly with Rsbuild v2.

Copilot uses AI. Check for mistakes.
@chenjiahan chenjiahan merged commit 5b7ac46 into main Jan 15, 2026
11 checks passed
@chenjiahan chenjiahan deleted the update_peers_0115 branch January 15, 2026 03:58
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