Development Workflow
Coding (Github)
Coding workflow:
- Target is at least 1 merged PR/day.
- Keep commits/PRs small (<500 lines of change) and do it often.
- Any PR should include the Linear link, unit/integration tests, and have CircleCI/CodeRabbit passing.
- Never push directly to master/main or staging or develop branches.
- We use the Git Flow deployment strategy.
API breaking changes:
- Discussed in #tech Slack channel.
- Separate API environment (Heroku Review Apps) created for FE to test/implement changes.
- API team coordinates PR merge with FE deployment.
Peer review for merging PRs (someone other than the requester):
- Please provide comments/screenshots/GIFs along with your PR so that the reviewer(s) can understand the intended functionality and context.
- For repos with CD (ex. Heroku auto deploy master), the PR should include the Linear issue link so that the reviewer can Deliver it.
Design
Development
Hosting