Please adhere to the coding style throughout the project.
Always use tabs. Here is a short explanation why tabs are preferred.
Also see: How to name things in programming
NOTE: A .editorconfig file is provided, if your editor supports it.
Keep the commit log as healthy as the code. It is one of the first places new contributors will look at the project.
- No more than one change per commit. There should be no changes in a commit which are unrelated to its message.
- Every commit should pass all tests on its own.
- Follow these conventions when writing the commit message
When filing a Pull Request, make sure it is rebased on top of most recent master. If you need to modify it or amend it in some way, you should always appropriately fixup the issues in git and force-push your changes to your fork.
Also see: Github Help: Using Pull Requests
You can always ask for help in our IRC channel, #Hearthsim on Freenode.