HaruhikoMotokawa PR Readiness Checklist
When evaluating a pull request, check each item in checklist.md and report:
- PASS — criterion is met
- FAIL — criterion is not met (include file:line reference)
- N/A — criterion does not apply to this change
Evaluation Process
- Read the PR diff and all changed files
- Check each item in
checklist.md - Report results in this format:
## PR Checklist Results: {pr-title}
| # | Item | Result | Notes |
|---|------|--------|-------|
| 1 | No TODO without issue ref | PASS | |
| 2 | New functionality has tests | FAIL | lib/auth/service.dart — no test for refreshToken() |
| 3 | README updated | N/A | Internal refactor only |
### Summary: X/Y criteria met
### Verdict: READY TO MERGE / NEEDS FIXES
Criteria Count
Always include the total number of applicable criteria and how many passed. A PR is READY TO MERGE only when all non-N/A items are PASS.