上一轮我说"剩余中文都合理",说得太满。这三处不合理: 1. bootable/linux-setup/test-installation.sh —— 在用户的 Linux 机器上跑的 自检脚本,输出中英混杂(check "安装目录存在" ... "Directory $INSTALL_DIR") 2. issue / PR 模板 —— 贡献者第一眼看到的东西,而 CONTRIBUTING.md 已经是 英文了。顺带修掉模板里指向上游官网和上游微信的链接(那些我们控制不了), 改为指向本仓库。PR 模板加了一条"说明你在哪个平台跑过" —— 只在 Mac 上 验证过的 .bat 改动一直是本项目最主要的故障来源。 3. release.yml 的发布说明正文 —— 那是用户在 release 页面读到的文字,内容 还停留在旧产品(DeepSeek/通义、预装微信 QQ 钉钉飞书企微)。 第 3 项里还有一条**和事实相反**的建议:它让用户把 U 盘格式化成 NTFS。 macOS 对 NTFS 只读,照做正好会让"配置跟着盘走"这个核心卖点在 Mac 上 彻底失效。改为 exFAT,并说明为什么慢、以及我们如何用本机缓存补偿。 track-upstream.yml 的输入描述和输出也翻了 —— 它们会显示给手动触发 workflow 的人。
This commit is contained in:
72
.github/PULL_REQUEST_TEMPLATE.md
vendored
72
.github/PULL_REQUEST_TEMPLATE.md
vendored
@@ -1,51 +1,53 @@
|
||||
<!--
|
||||
感谢贡献 U-Claw!请按以下模板填写,方便维护者审阅。
|
||||
空白或仅有标题的 PR 通常会被关闭,请勿提交无说明的改动。
|
||||
Thanks for sending this.
|
||||
|
||||
A PR with a title and no description usually gets closed — not out of
|
||||
strictness, but because nobody can tell whether it is safe to merge.
|
||||
-->
|
||||
|
||||
## 改动类型 / Type
|
||||
## What this changes
|
||||
|
||||
<!-- 勾选适用的([x]) -->
|
||||
<!-- One or two sentences. What is different afterwards? -->
|
||||
|
||||
- [ ] 🐛 Bug 修复
|
||||
- [ ] ✨ 新功能
|
||||
- [ ] 📝 文档/README
|
||||
- [ ] 🔧 脚本/构建(portable, install, bootable, u-claw-app)
|
||||
- [ ] 🎨 技能 (skills/)
|
||||
- [ ] ♻️ 重构(不改变行为)
|
||||
- [ ] 🧪 测试
|
||||
## Why
|
||||
|
||||
## 改动了什么 / What
|
||||
<!-- The problem it solves. Link an issue if there is one. -->
|
||||
|
||||
<!-- 一两句话说清楚这个 PR 做了什么。不要复制 git diff。 -->
|
||||
## Type
|
||||
|
||||
## 为什么 / Why
|
||||
- [ ] 🐛 Fix
|
||||
- [ ] ✨ New capability
|
||||
- [ ] 📝 Docs
|
||||
- [ ] 🎨 Skill content (`skills/`)
|
||||
- [ ] ♻️ Refactor — no behaviour change
|
||||
- [ ] 🔧 Build, CI or tooling
|
||||
|
||||
<!-- 哪个 issue / 哪个使用场景驱动了这次改动?关联 issue: #xxx -->
|
||||
## Which builds this touches
|
||||
|
||||
## 测试方式 / How to verify
|
||||
- [ ] `portable/` — the USB build
|
||||
- [ ] `install/` — one-line install
|
||||
- [ ] `bootable/` — Linux USB
|
||||
- [ ] Shared (`lib/`, `skills/`, tests)
|
||||
|
||||
## How you tested it
|
||||
|
||||
<!--
|
||||
告诉维护者怎么验证。比如:
|
||||
- 在 macOS Apple Silicon 上跑 `bash install/install.sh`,安装成功
|
||||
- 双击 `portable/Mac-Start.command`,能正常打开 dashboard
|
||||
- Windows 11 PowerShell 跑 `irm https://u-claw.org/install.ps1 | iex`,安装成功
|
||||
Not decoration. Say what you actually ran.
|
||||
|
||||
node --test # NOT `node --test tests/` — that runs nothing
|
||||
|
||||
And for anything touching a launcher or an installer, say which platform you
|
||||
ran it on. A change to a .bat verified only on a Mac is worth saying so about —
|
||||
that has been the main source of breakage here.
|
||||
-->
|
||||
|
||||
## 影响范围 / Scope
|
||||
- [ ] `node --test` passes
|
||||
- [ ] Ran it on: <!-- macOS / Windows / Linux -->
|
||||
|
||||
<!-- 勾选受影响的模块 -->
|
||||
## Checklist
|
||||
|
||||
- [ ] portable/ (便携 USB)
|
||||
- [ ] u-claw-app/ (Electron 桌面)
|
||||
- [ ] bootable/ (Linux 可启动 U 盘)
|
||||
- [ ] install/ (一键安装脚本)
|
||||
- [ ] skills/ (技能内容与 manifest)
|
||||
- [ ] 仅文档
|
||||
|
||||
## 自检 / Checklist
|
||||
|
||||
- [ ] 我已经在本地实际跑过相关脚本/功能
|
||||
- [ ] 没有提交 `node_modules/`、`app/`、`data/` 等运行时产物
|
||||
- [ ] 没有把任何 API Key、Token 写进文件
|
||||
- [ ] commit message 能说清楚改了什么
|
||||
- [ ] No `node_modules/`, `app/`, `data/`, `*.dmg`, `*.exe` in the diff
|
||||
- [ ] No API keys, tokens or passwords in any file
|
||||
- [ ] User-facing text is in `lib/messages/*.json`, not hardcoded in a script
|
||||
- [ ] `.bat` files stay pure ASCII with CRLF; `.command` files stay LF-only
|
||||
- [ ] Unrelated files are not reformatted
|
||||
|
||||
Reference in New Issue
Block a user