-
-
Notifications
You must be signed in to change notification settings - Fork 138
Contributing
Austin Bischoff edited this page Feb 27, 2015
·
4 revisions
Contributions are welcome and will be fully credited.
-
Document any change in behavior - Make sure the
README.md
and any other relevant documentation are kept up-to-date. -
Create feature branches - Don't ask us to pull from your master branch.
-
One pull request per feature - If you want to do more than one thing, send multiple pull requests.
-
We use the PSR-2 Coding Standard
-
Use the following commands to test your code before submitting it:
$ vendor/bin/phpcs src tests --extensions=php --ignore=bootstrap --report=checkstyle --report-file=build/logs/checkstyle.xml --standard=build/config/phpcs.xml -v
$ vendor/bin/phpmd src,tests xml build/config/phpmd.xml
-
Add tests! - Your patch won't be accepted if it doesn't have tests.
-
Run tests by calling
phpunit
$ vendor/bin/phpunit
Both coding standard and tests are validated by Travis CI.
If you do not feel comfortable writing your own changes feel free open up a new issue for to add a game or feature.