First of all, thank you for contributing, you are awesome!
Here are few rules to follow for a easier code review before the maintainers accept and merge your request.
- Run the test suite.
- Write (or update) unit tests.
- Write (or update) documentation.
- Write commit messages that make sense
- Rebase your branch before submitting your Pull Request.
- [Squash your commits] to "clean" your pull request before merging (http://gitready.com/advanced/2009/02/10/squashing-commits-with-rebase.html).
- Write and good description which gives the context and/or explains why you are creating it.
Thank you!
Get sources dependencies :
composer install
Launch unit tests with Atoum
./bin/atoum
Check style use Coke
Start code analysis :
./bin/coke