Compare commits
2 Commits
issue-8-fi
...
issue-4-re
| Author | SHA1 | Date | |
|---|---|---|---|
|
a2c77a338b
|
|||
|
37a882915f
|
@@ -22,7 +22,7 @@ You will receive a PR number to review. Follow this process:
|
|||||||
- **Test Coverage**: Missing tests, untested edge cases
|
- **Test Coverage**: Missing tests, untested edge cases
|
||||||
3. Generate a structured review comment
|
3. Generate a structured review comment
|
||||||
4. Post the review using `tea comment <number> "<review body>"`
|
4. Post the review using `tea comment <number> "<review body>"`
|
||||||
5. **If verdict is LGTM**: Approve with `tea pulls approve <number>`, then auto-merge with `tea pulls merge <number> --style rebase`
|
5. **If verdict is LGTM**: Merge with `tea pulls merge <number> --style rebase`
|
||||||
6. **If verdict is NOT LGTM**: Do not merge; leave for the user to address
|
6. **If verdict is NOT LGTM**: Do not merge; leave for the user to address
|
||||||
|
|
||||||
## Review Comment Format
|
## Review Comment Format
|
||||||
|
|||||||
@@ -20,3 +20,19 @@ Ask the user what action to take:
|
|||||||
- **Merge**: Post review summary as comment, then merge with rebase style
|
- **Merge**: Post review summary as comment, then merge with rebase style
|
||||||
- **Request changes**: Leave feedback without merging
|
- **Request changes**: Leave feedback without merging
|
||||||
- **Comment only**: Add a comment for discussion
|
- **Comment only**: Add a comment for discussion
|
||||||
|
|
||||||
|
## Merging
|
||||||
|
|
||||||
|
Always use tea CLI for merges to preserve user attribution:
|
||||||
|
|
||||||
|
```bash
|
||||||
|
tea pulls merge <number> --style rebase
|
||||||
|
```
|
||||||
|
|
||||||
|
For review comments, use `tea comment` since `tea pulls review` is interactive-only:
|
||||||
|
|
||||||
|
```bash
|
||||||
|
tea comment <number> "<review summary>"
|
||||||
|
```
|
||||||
|
|
||||||
|
> **Warning**: Never use the Gitea API with admin credentials for user-facing operations like merging. This causes the merge to be attributed to the admin account instead of the user.
|
||||||
|
|||||||
Reference in New Issue
Block a user