Skip to content

Latest commit

 

History

History
27 lines (17 loc) · 1.34 KB

CONTRIBUTING.md

File metadata and controls

27 lines (17 loc) · 1.34 KB

Coding style

This project follows the HearthSim Styleguide.

In short:

  1. Always use tabs. Here is a short explanation why tabs are preferred.
  2. Always use double quotes for strings, unless single quotes avoid unnecessary escapes.
  3. When in doubt, PEP8. Follow its naming conventions.
  4. Know when to make exceptions.

Also see: How to name things in programming

Commits and Pull Requests

Keep the commit log as healthy as the code. It is one of the first places new contributors will look at the project.

  1. No more than one change per commit. There should be no changes in a commit which are unrelated to its message.
  2. Every commit should pass all tests on its own.
  3. 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