Loading...
Loading...
Compare original and translation side by side
"Read with intent. Question with purpose. Document with care."
"带着目的阅读,带着疑问审视,用心记录。"
READ → NOTE ISSUES → CLASSIFY → REPORT阅读 → 记录问题 → 分类 → 提交报告| Level | Symbol | Meaning | Action |
|---|---|---|---|
| Blocking | 🚫 | Must fix before merge | Request changes |
| Important | ⚠️ | Should fix or explain | Request changes |
| Minor | 💡 | Nice to fix | Comment only |
| Praise | 🎉 | Good work! | Celebrate |
| 级别 | 标识 | 含义 | 处理方式 |
|---|---|---|---|
| 阻塞性 | 🚫 | 合并前必须修复 | 请求修改 |
| 重要 | ⚠️ | 应该修复或作出解释 | 请求修改 |
| 次要 | 💡 | 建议修复 | 仅评论 |
| 表扬 | 🎉 | 做得好! | 给予肯定 |
review:
name: "PR #123: Add user authentication"
status: "in_progress"
findings:
blocking:
- id: "B1"
file: "src/auth/login.ts"
line: 45
type: "security"
summary: "Timing attack vulnerability"
important: []
minor: []
praise: []
verification:
tests: { ran: true, passed: true }
linter: { ran: true, passed: false, issues: 3 }review:
name: "PR #123: Add user authentication"
status: "in_progress"
findings:
blocking:
- id: "B1"
file: "src/auth/login.ts"
line: 45
type: "security"
summary: "Timing attack vulnerability"
important: []
minor: []
praise: []
verification:
tests: { ran: true, passed: true }
linter: { ran: true, passed: false, issues: 3 }tests:
- "npm test"
- "pytest"
- "go test ./..."
linters:
- "npm run lint"
- "flake8"
- "golangci-lint run"tests:
- "npm test"
- "pytest"
- "go test ./..."
linters:
- "npm run lint"
- "flake8"
- "golangci-lint run"| Outcome | Meaning |
|---|---|
| Good to merge |
| Has blocking/important issues |
| Minor feedback only |
| 结果 | 含义 |
|---|---|
| 可以合并 |
| 存在阻塞性/重要问题 |
| 仅次要反馈 |