Conversation
✅ Deploy Preview for rstest-dev ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
There was a problem hiding this comment.
Pull Request Overview
This PR adds documentation for the --reporter CLI option to the Rstest documentation. The changes update both English and Chinese documentation to include the new CLI option and simplify the reporter description.
- Added
--reporteroption to CLI documentation tables in both languages - Simplified the description of the reporters configuration by removing redundant text
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| website/docs/zh/guide/basic/cli.mdx | Added --reporter option to Chinese CLI options table |
| website/docs/zh/config/test/reporters.mdx | Simplified Chinese reporter description |
| website/docs/en/guide/basic/cli.mdx | Added --reporter option to English CLI options table |
| website/docs/en/config/test/reporters.mdx | Simplified English reporter description |
| | `--testTimeout <value>` | 设置单个测试的超时时间（毫秒），详见 [testTimeout](/config/test/testTimeout) | | ||
| | `--hookTimeout <value>` | 设置单个测试 hook 的超时时间（毫秒），详见 [hookTimeout](/config/test/hookTimeout) | | ||
| | `--retry <retry>` | 测试失败时重试次数，详见 [retry](/config/test/retry) | | ||
| | `--reporter <reporter>` | 指定测试报告器，详见 [reporters](/config/test/reporters) | |
There was a problem hiding this comment.
[nitpick] The --reporter option is placed inconsistently in the table. It should be positioned after --retry to maintain logical grouping with other test configuration options, rather than being inserted between --retry and --maxConcurrency.
| | `-r, --root <root>` | Specify the project root directory, see [root](/config/test/root) | | ||
| | `--globals` | Provide global APIs, see [globals](/config/test/globals) | | ||
| | `--isolate` | Run tests in an isolated environment, see [isolate](/config/test/isolate) | | ||
| | `--reporter <reporter>` | Specify the test reporter, see [reporters](/config/test/reporters) | |
There was a problem hiding this comment.
[nitpick] The --reporter option placement is inconsistent between the English and Chinese documentation. In the English version, it's placed after --isolate, while in the Chinese version it's after --retry. Consider maintaining consistent ordering across both language versions.
| | `--reporter <reporter>` | Specify the test reporter, see [reporters](/config/test/reporters) | |
Summary
add
--reporteroption.Related Links
Checklist