- Pull requests must contain a succinct, clear summary of what the user need is driving this feature change.
- Follow our Git styleguide
- Make a feature branch
- Ensure your branch contains logical atomic commits before sending a pull request - follow our Git styleguide
- Pull requests are automatically integration tested, where applicable using Travis CI, which will report back on whether the tests still pass on your branch
- You may rebase your branch after feedback if it's to include relevant updates from the master branch. We prefer a rebase here to a merge commit as we prefer a clean and straight history on master with discrete merge commits for features
- Follow the style guide
- URLs should use hyphens, not underscores
- Must be readable with meaningful naming, eg no short hand single character variable names
- Follow our Ruby style guide
Write tests.
Make sure the regression tests are passing.