We love pull requests. Here's a quick guide:
-
Fork the repo.
-
Run the tests. We only take pull requests with passing tests, and it's great to know that you have a clean slate:
bundle && bundle exec rake
-
Add a test for your change. Only refactoring and documentation changes require no new tests. If you are adding functionality or fixing a bug, we need a test!
-
Make the test pass. Always use
sample
,shuffle
, andrand
from the Base class (just like the rest of the code) rather thanArray#sample
,Array#shuffle
andKernel#rand
to preserve the deterministic feature. -
Push to your fork and submit a pull request.
Syntax:
- Two spaces, no tabs.
- No trailing whitespace. Blank lines should not have any space.
- Prefer
&&
,||
overand
,or
. MyClass.my_method(my_arg)
notmy_method( my_arg )
ormy_method my_arg
.a = b
and nota=b
.- Follow the conventions you see used in the source already.