Skip to main content

Pull Requests

A pull request should be small, focused, and easy to review.

Rules​

  • One PR per issue — do not bundle multiple issues into one PR
  • A PR should touch 3 to 8 files at most
  • All linting and formatting checks must pass before requesting review
  • Do not open a PR against any branch other than main
  • Do not merge your own PR without approval

PR Title Format​

feat: add employment history section to profile view
fix: resolve toggle not disabling end date field

PR Description Template​

Use this template for every PR:

## What does this PR do?
Brief description of the change.

## Related Issue
Closes #<issue number>

## Screenshots
Add a screenshot or screen recording if the change affects UI.

## Checklist
- [ ] Follows coding guidelines
- [ ] One issue per PR
- [ ] Uses existing shadcn/ui components only
- [ ] Mobile responsive
- [ ] No console errors
- [ ] pnpm lint passes
- [ ] pnpm prettier:write has been run
- [ ] pnpm build passes with no errors

Before Opening a PR​

pnpm prettier:write
pnpm lint
pnpm build

Review Process​

  • PRs are reviewed by the maintainer before merging
  • Address all review comments before requesting a re-review
  • Do not merge your own PR without approval