Milk-V Pioneer: RevyOS Test Report#381
Closed
ChestNutICE wants to merge 3 commits into
Closed
Conversation
Signed-off-by: ChestNutICE <sudoavocado@gmail.com> LicheeRVDock:DebianTrixieReport Signed-off-by: ChestNutICE <sudoavocado@gmail.com>
20251030 TestReport
Reviewer's GuideIntroduces a comprehensive, bilingual test report for the RevyOS Pioneer platform, detailing environment setup, installation steps, test results, and logs in both English and Chinese. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
![sourcery-ai[bot]](https://avatars.githubusercontent.com/in/48477?s=60&v=4){kind=small}
There was a problem hiding this comment.
Hey there - I've reviewed your changes - here's some feedback:
- Align the metadata sys_ver (20251030) with the "Operating System Information" section (currently listing 20241230) to avoid version confusion.
- Fix typos and grammar in the install steps (e.g., change “fist-time” to “first-time” and “needs to update your firmware” to “need to update your firmware”).
- Consider adding checksum (e.g., SHA256) verification instructions for downloaded firmware and images to ensure file integrity.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- Align the metadata sys_ver (20251030) with the "Operating System Information" section (currently listing 20241230) to avoid version confusion.
- Fix typos and grammar in the install steps (e.g., change “fist-time” to “first-time” and “needs to update your firmware” to “need to update your firmware”).
- Consider adding checksum (e.g., SHA256) verification instructions for downloaded firmware and images to ensure file integrity.
## Individual Comments
### Comment 1
<location> `Pioneer/RevyOS/README_20251030.md:32` </location>
<code_context>
+
+### Flashing Firmware
+
+#### SD card (suggest for fist-time)
+
+Get the firmware and flash to the SD card:
</code_context>
<issue_to_address>
**issue (typo):** Typo: 'fist-time' should be 'first-time'.
```suggestion
#### SD card (suggest for first-time)
```
</issue_to_address>
### Comment 2
<location> `Pioneer/RevyOS/README_20251030.md:66` </location>
<code_context>
+
+```bash
+zstd -d /path/to/revyos-pioneer-20251030-004123.img.zst
+dd if=/path/to/revyos-pioneer-20251030-004123.img.zst of=/dev/yout-device bs=4M status=progress
+```
+
</code_context>
<issue_to_address>
**issue (typo):** Typo: 'yout-device' should be 'your-device'.
Update the dd command to use '/dev/your-device' instead of '/dev/yout-device'.
```suggestion
dd if=/path/to/revyos-pioneer-20251030-004123.img.zst of=/dev/your-device bs=4M status=progress
```
</issue_to_address>
### Comment 3
<location> `Pioneer/RevyOS/README_20251030_zh.md:16` </location>
<code_context>
+
+- Milk-V Pioneer Box v1.3
+- microSD 卡一张
+- NMVE SSD 及硬盘盒
+- HDMI 线 + 显示器
+
</code_context>
<issue_to_address>
**issue (typo):** Typo: 'NMVE' should be 'NVME'.
Update the spelling to 'NVME' to ensure technical accuracy.
```suggestion
- NVME SSD 及硬盘盒
```
</issue_to_address>
### Comment 4
<location> `Pioneer/RevyOS/README_20251030_zh.md:58` </location>
<code_context>
+
+```bash
+zstd -d /path/to/revyos-pioneer-20251030-004123.img.zst
+dd if=/path/to/revyos-pioneer-20251030-004123.img.zst of=/dev/yout-device bs=4M status=progress
+```
+
</code_context>
<issue_to_address>
**issue (typo):** Typo: 'yout-device' should be 'your-device'.
Update the dd command to use '/dev/your-device' instead of '/dev/yout-device'.
```suggestion
dd if=/path/to/revyos-pioneer-20251030-004123.img.zst of=/dev/your-device bs=4M status=progress
```
</issue_to_address>Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
|
|
||
| ### Flashing Firmware | ||
|
|
||
| #### SD card (suggest for fist-time) |
There was a problem hiding this comment.
issue (typo): Typo: 'fist-time' should be 'first-time'.
Suggested change
| #### SD card (suggest for fist-time) | |
| #### SD card (suggest for first-time) |
|
|
||
| ```bash | ||
| zstd -d /path/to/revyos-pioneer-20251030-004123.img.zst | ||
| dd if=/path/to/revyos-pioneer-20251030-004123.img.zst of=/dev/yout-device bs=4M status=progress |
There was a problem hiding this comment.
issue (typo): Typo: 'yout-device' should be 'your-device'.
Update the dd command to use '/dev/your-device' instead of '/dev/yout-device'.
Suggested change
| dd if=/path/to/revyos-pioneer-20251030-004123.img.zst of=/dev/yout-device bs=4M status=progress | |
| dd if=/path/to/revyos-pioneer-20251030-004123.img.zst of=/dev/your-device bs=4M status=progress |
|
|
||
| - Milk-V Pioneer Box v1.3 | ||
| - microSD 卡一张 | ||
| - NMVE SSD 及硬盘盒 |
There was a problem hiding this comment.
issue (typo): Typo: 'NMVE' should be 'NVME'.
Update the spelling to 'NVME' to ensure technical accuracy.
Suggested change
| - NMVE SSD 及硬盘盒 | |
| - NVME SSD 及硬盘盒 |
|
|
||
| ```bash | ||
| zstd -d /path/to/revyos-pioneer-20251030-004123.img.zst | ||
| dd if=/path/to/revyos-pioneer-20251030-004123.img.zst of=/dev/yout-device bs=4M status=progress |
There was a problem hiding this comment.
issue (typo): Typo: 'yout-device' should be 'your-device'.
Update the dd command to use '/dev/your-device' instead of '/dev/yout-device'.
Suggested change
| dd if=/path/to/revyos-pioneer-20251030-004123.img.zst of=/dev/yout-device bs=4M status=progress | |
| dd if=/path/to/revyos-pioneer-20251030-004123.img.zst of=/dev/your-device bs=4M status=progress |
Collaborator
|
KevinMX
requested changes
Oct 30, 2025
|
|
||
| ### Operating System Information | ||
|
|
||
| - System Version: RevyOS 20241230 |
Collaborator
There was a problem hiding this comment.
Suggested change
| - System Version: RevyOS 20241230 | |
| - System Version: RevyOS 20251030 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Checklist
Summary by Sourcery
Add detailed test report documentation for RevyOS Pioneer 20251030 in English and Chinese
Documentation: