This guide covers the basics of how to contribute to the Whitehall project.
Ruby code should follow the rules below and also follow the testing guideline s.
Frontend code should follow the css and javascript guidelines.
- Make a branch
- Work in any way you like
- Rebase your branch into logical commits before sending a pull request - follow our Git styleguide
- If you're working off the Pivotal backlog, include the ticket number in the pull request details
- Pull requests are automatically integration tested using Jenkins, 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 include relevant updates to 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
- Someone must review your code and give you a "thumbs up"
- Someone else must product/design review your changes and give you another "thumbs up"
- Merge your PR and delete the branch
- Titles and navigation links should only capitalise first letter, not every word.
- URLs should use hyphens, not underscores.
- Follow the style guide.
- Don't commit additional whitespace
- Use ruby 1.9.2 hash syntax wherever possible
- Models should have:
- All associations at the top
- Then any scopes
- Then validations
- Then code
- Follow the GOV.UK Ruby styleguide