Pull Requests and Community Contributions are the bread and butter of open source software. Every contribution- from bug reports to feature requests, typos to full new features- are greatly appreciated.
-
One Item Per Pull Request or Issue. This makes it much easier to review code and merge it back in, and prevents issues with one request from blocking another.
-
Code Coverage is extremely important, and pull requests are much more likely to be accepted if testing is also improved. New code should be properly tested, and all tests must pass.
-
Read the LICENSE document and make sure you understand it, because your code is going to be released under it.
-
Be prepared to make revisions. Don't be discouraged if you're asked to make changes, as that is just another step towards refining the code and getting it merged back in.
-
Remember to add the relevant documentation, particular the docblock comments.
This project follows the PSR standards set forth by the PHP Framework Interop Group.
All code most follow these standards to be accepted. The easiest way to accomplish this is to run php-cs-fixer once the new changes are finished. The php-cs-fixer package is installed as a development dependency of this project.
composer install --dev
vendor/bin/php-cs-fixer fix ./
First install dependencies using Composer. It's important to include the dev packages:
composer install --dev
The "runTests.sh" script runs the full test suite- phpunit, php-cs-fixer, as well as any environmental setup:
tests/runTests.sh
To call phpunit directly:
vendor/bin/phpunit
To call php-cs-fixer directly:
vendor/bin/php-cs-fixer fix ./ -vv --dry-run