PR Guide
PR Checklist
Section titled “PR Checklist”Before submitting a PR, verify the following:
-
pnpm run buildcompletes successfully - No Markdown syntax errors
- Spelling and grammar reviewed
- Image/link paths verified to work
- For assignment submissions: confirm path is
assignments/[lab/week]-XX/[student-id]/
Commit Message Convention
Section titled “Commit Message Convention”<type>: <description>
# Examplesfeat: add HOTL diagram to Week 1 lecture notesfix: fix MIG code example error in Week 3docs: clarify Lab 04 submission requirementschore: update dependenciesTypes:
feat: New content addedfix: Error correcteddocs: Documentation improvedstyle: Formatting change (no content change)chore: Other maintenance tasks
Assignment Submission PR
Section titled “Assignment Submission PR”# PR title format[Submit] Lab 04 - 20230001 Hong Gildong
# PR body example## Submission Contents- harness.sh implementation (ralph loop + backpressure)- PROMPT.md written (3 tasks)- pytest-based test suite
## Execution Results- Loop log showing success after 2 failures attached- Garbage collection behavior confirmedReview Process
Section titled “Review Process”- PR created → GitHub Actions automated build check
- Build passes → instructor/peer review
- Review approved → merge → automatic deployment
Responding to Review Comments
Section titled “Responding to Review Comments”When a reviewer requests changes:
# After making fixesgit add .git commit -m "fix: address review - fix XXX"git push origin [branch-name]# PR updates automaticallyCommon Mistakes
Section titled “Common Mistakes”| Mistake | Fix |
|---|---|
| Build fails | Run pnpm run build locally and check errors |
| Branch is behind main | Run git pull upstream main then rebase |
| Wrong assignment path | Re-check assignments/lab-XX/[student-id]/ format |