Development Process & Standards

Overview

We're doing Scrum-lite. Sprints start on Monday.

All times are US/Eastern

Sprint Board

The sprint board is here. Install the Zenhub extension if you can't see it.

Protocol team standups are at 10am, app team standups are at 11am.

Definition of "Done"

• required work is done
• tests for new work are written and pass
• unit/functional/integration/regression tests pass
• finished work was tested manually
• its secure
• documentation is written/updated
• code is merged into master
• code is ready to go live
• team agrees item is done

Dev Process

1. assign issue to yourself, mark it as In Progress
2. code code code commit code test code commit test feedback code test ...
   1. n.b - a WIP pull request can be created here to solicit feedback
3. squash/rebase code for sharing, write quality commit messages
4. run automated linting/testing suite. if there are issues, goto 1
5. test the code yourself (on every OS/platform/environment we support)
6. document your changes in the changelog
7. create PR (or update existing PR)
8. assign someone to review and approve your PR. if reviewer raises any issues, goto 1
9. squash fixup commits; merge your PR into master; delete branch
10. close issue

Code Standards

0. the maxims
1. Every commit in master should work and pass tests/linting
2. See Branching and Merging for how to git
3. Commit messages should be informative, aimed at developers, and provide context/explanation for the change being made
4. Commits on master should stand-alone
5. Force pushes on feature branches are allowed; please commit early and often, but clean-up the mess before merging into master.
6. Pull requests should have an informative changelog entry (see changelog format). The changelog is aimed at downstream developers and technical users. This is slightly different from commit messages.

Code Review

1. Reviewing code takes priority over writing code. Code waiting for review means that somebody else is blocked. Be a good teammate. Also, code that is written but not merged is wasted inventory and expensive.