👍🎉 Many thanks for taking the time to contribute! 🎉👍
When you contribute, please take the following things into account:
Please note that this project is released with a Contributor Code of Conduct. By participating in this project, you agree to abide by its terms.
We have provided an issue template that will help you create helpful tickets.
- … you are asking how to use some feature. Please use the phpList community for this purpose.
- … your issue is about a security vulnerability. Please contact us directly to report security issues.
Before you report an issue, please search through the existing issues here on GitHub to see if your issue is already reported or fixed to make sure you are not reporting a duplicated issue.
Also please make sure you have the latest version of this package and check if the issue still exists.
Third-party contributions are essential for keeping the project great.
We want to keep it as easy as possible to contribute changes that get things working in your environment.
There are a few guidelines that we need contributors to follow so that we can have a chance of keeping on top of things:
- Make sure you have a GitHub account.
- Fork this Git repository.
- Clone your forked repository and install the development dependencies doing
a
composer install. - Add a local remote "upstream" so you will be able to synchronize your fork with the original repository.
- Create a local branch for your changes.
- Add unit tests for your changes (if your changes are code-related). These tests should fail without your changes.
- Add your changes. Your added unit tests now should pass, and no other tests should be broken. Check that your changes follow the coding style.
- Add a changelog entry.
- Commit and push your changes.
- Create a pull request for your changes. Check that the Travis build is green. (If it is not, fix the problems listed by Travis.) We have provided a template for pull requests as well.
- Request a review.
- Together with your reviewer, polish your changes until they are ready to be merged.
Please cover all changes with automatic tests and make sure that your code does not break any existing tests. We will only merge pull request that include full code coverage of the fixed bugs and the new features.
For being able to run the integration tests, you will need a local MySQL database and a user with access permissions to that database.
After you have created the database and the user, please import the database
schema once. Assuming that your database is named phplist_test, the user is
named phplist, and the password is batterystaple, the command looks like
this:
mysql -u phplist_test --password=batterystaple phplist_test < vendor/phplist/core/resources/Database/Schema.sqlWhen running the integration tests, you will need to specify the database name and access credentials on the command line (in the same line):
PHPLIST_DATABASE_NAME=phplist_test PHPLIST_DATABASE_USER=phplist PHPLIST_DATABASE_PASSWORD=batterystaple vendor/bin/phpunit tests/Integration/Please make your code clean, well-readable and easy to understand.
Please use the same coding style (PSR-2) as the rest of the code. Indentation for all files is four spaces.
We will only merge pull requests that follow the project's coding style.
Please check your code with the provided PHP_CodeSniffer standard:
vendor/bin/phpcs --standard=vendor/phplist/core/config/PhpCodeSniffer/ src/ tests/Please also check the code structure using PHPMD:
vendor/bin/phpmd src/ text vendor/phplist/core/config/PHPMD/rules.xmlAnd also please run the static code analysis:
vendor/bin/phpstan analyse -l 5 src/ tests/You can also run all code style checks using one long line from a bash shell:
find src/ tests/ -name '*.php' -print0 | xargs -0 -n 1 -P 4 php -l && vendor/bin/phpstan analyse -l 5 src/ tests/ && vendor/bin/phpmd src/ text vendor/phplist/core/config/PHPMD/rules.xml && vendor/bin/phpcs --standard=vendor/phplist/core/config/PhpCodeSniffer/ src/ tests/This will execute all tests except for the unit tests and the integration tests.