First of all, thank you for contributing, you are awesome!
This project uses the fork & pull model of development. This means that in order to contribute you need to submit a pull request.
Please make an issue about what you intent to make before you create the pull request and reference that issue from your pull request.
Here are a few rules to follow in order to ease code reviews and discussions before maintainers accept and merge your work:
Please rebase your branch
before submitting your Pull Request. One may ask you to squash your
commits
too. This is used to "clean" your Pull Request before merging it (we don't want
commits such as fix tests
, fix 2
, fix 3
, etc.).
We use semver, please take that into account. Randomly breaking BC is not an option. Please use the Symfony BC promise as a guideline of we consider BC.
You MUST follow the PSR-1, PSR-2.
To fix your code according to the project standards, you can run PHP-CS-Fixer tool before commit:
php-cs-fixer fix --level psr2 .
- You MUST run the test suite.
- You SHOULD write (or update) unit tests for any other non-trivial functionality.
Test suite can be run using the following command:
vendor/bin/phpunit
You SHOULD write documentation for the code you add.
Also, please, write commit messages that make sense. While creating your Pull Request on GitHub, you MUST write a description which gives the context and/or explains why you are creating it.
Thank you!