Skip to content

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.

Development Tools

Design

Development

Hosting