-
Notifications
You must be signed in to change notification settings - Fork 69
feat: add metal case for cubie a5e #834
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Other comments (1)
-
docs/accessories/metal-case-cubie-a5e.md (41-41)
The quantity format for M2*16.6 mm copper posts is inconsistent with the other quantity formats in the list. Please standardize:
- M2\*16.6 mm 铜柱 x 4(不搭配 PoE HAT 使用:固定主板)
💡 To request another review, post a new comment with "/windsurf-review".
|
||
- **Custom-fit for CUBIE A5E** - Precise cutouts and structural design ensure perfect compatibility with all interfaces and expansion capabilities. | ||
|
||
- **Extremely Compact Design** - Case dimensions approximately 73 mm _ 64 mm _ 30mm, lightweight and portable, about the size of a palm, perfect for deployment in tight spaces. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The dimensions in the "Extremely Compact Design" section are formatted incorrectly. The underscores (_
) are being used as multiplication symbols, but in Markdown they cause text to be italicized. Consider using proper multiplication symbols (×) or 'x' instead.
- **Extremely Compact Design** - Case dimensions approximately 73 mm _ 64 mm _ 30mm, lightweight and portable, about the size of a palm, perfect for deployment in tight spaces. | |
- **Extremely Compact Design** - Case dimensions approximately 73 mm × 64 mm × 30mm, lightweight and portable, about the size of a palm, perfect for deployment in tight spaces. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
重新提交了commit进行修改
|
||
### 不搭配 PoE HAT 使用 | ||
|
||
#### Step 1: 安装导热硅胶 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There's an inconsistency in the step numbering format between the two installation methods. For better consistency, please standardize the format:
#### Step 1: 安装导热硅胶 | |
#### Step 1: 安装导热硅胶 |
And similarly for all other steps in both installation methods.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
重新提交了commit进行修改
Co-authored-by: windsurf-bot[bot] <189301087+windsurf-bot[bot]@users.noreply.github.com>
Description of changes