Skip to content

Conversation

@9aoy
Copy link
Contributor

@9aoy 9aoy commented Nov 25, 2025

Summary

add upgrade-rstest guide

Related Links

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

Copilot AI review requested due to automatic review settings November 25, 2025 03:54
@netlify
Copy link

netlify bot commented Nov 25, 2025

Deploy Preview for rstest-dev ready!

Name Link
🔨 Latest commit c7f27e7
🔍 Latest deploy log https://app.netlify.com/projects/rstest-dev/deploys/692528730a1d3400088421a5
😎 Deploy Preview https://deploy-preview-714--rstest-dev.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.

Copy link
Contributor

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 PR adds a new documentation guide for upgrading Rstest dependencies in both English and Chinese. The guide introduces users to Taze, a CLI tool for updating npm dependencies, and provides practical examples of how to upgrade Rstest packages using various Taze options. This documentation helps users keep their Rstest dependencies up to date, which is particularly important given that Rstest is still in the 0.x version stage where API changes can occur frequently.

  • Added comprehensive upgrade guide using the Taze CLI tool
  • Documented common upgrade scenarios including monorepo and locked version handling
  • Included both English and Chinese translations for international audience

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.

File Description
website/docs/en/guide/basic/upgrade-rstest.mdx New English documentation page explaining how to upgrade Rstest dependencies using Taze
website/docs/zh/guide/basic/upgrade-rstest.mdx New Chinese documentation page with the same upgrade guide content
website/docs/en/guide/basic/_meta.json Added "upgrade-rstest" to the navigation menu for English docs
website/docs/zh/guide/basic/_meta.json Added "upgrade-rstest" to the navigation menu for Chinese docs

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

You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.

Comment on lines +60 to +63
- To upgrade to a major version:

```bash
npx taze major --include /(rstest|rsbuild)/ -w
Copy link

Copilot AI Nov 25, 2025

Choose a reason for hiding this comment

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

[nitpick] This "To upgrade to a major version" example shows the exact same command as the initial usage example on line 20. This redundancy may confuse readers. Consider either removing this duplicate example or demonstrating a different use case (e.g., upgrading to a minor version without the major parameter).

Suggested change
- To upgrade to a major version:
```bash
npx taze major --include /(rstest|rsbuild)/ -w
- To upgrade to a minor or patch version (for packages that support it):
```bash
npx taze --include /(rstest|rsbuild)/ -w

Copilot uses AI. Check for mistakes.
Comment on lines +62 to +65
- 升级到 major 版本:

```bash
npx taze major --include /(rstest|rsbuild)/ -w
Copy link

Copilot AI Nov 25, 2025

Choose a reason for hiding this comment

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

[nitpick] This "升级到 major 版本" example shows the exact same command as the initial usage example on line 20. This redundancy may confuse readers. Consider either removing this duplicate example or demonstrating a different use case (e.g., upgrading to a minor version without the major parameter).

Suggested change
- 升级到 major 版本
```bash
npx taze major --include /(rstest|rsbuild)/ -w
- 升级到 minor 版本(不带 `major` 参数,仅升级到最新的 minor 或 patch 版本)
```bash
npx taze minor --include /(rstest|rsbuild)/ -w

Copilot uses AI. Check for mistakes.
Comment on lines +24 to +26

Rstest 尚未达到 1.0.0 版本,因此你需要在更新时添加 `major` 参数。

Copy link

Copilot AI Nov 25, 2025

Choose a reason for hiding this comment

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

[nitpick] The :::tip block has unnecessary blank lines (lines 24 and 26). Based on the codebase convention seen in other files, single-paragraph admonition blocks should not have blank lines inside. This should match the English version format:

:::tip
Rstest 尚未达到 1.0.0 版本,因此你需要在更新时添加 `major` 参数。
:::
Suggested change
Rstest 尚未达到 1.0.0 版本,因此你需要在更新时添加 `major` 参数。
Rstest 尚未达到 1.0.0 版本,因此你需要在更新时添加 `major` 参数。

Copilot uses AI. Check for mistakes.
@9aoy 9aoy merged commit d829679 into main Nov 25, 2025
32 of 33 checks passed
@9aoy 9aoy deleted the docs/update-rstest branch November 25, 2025 04:04
@9aoy 9aoy mentioned this pull request Nov 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants