Etiquette
This project is open source, and as such, the maintainers give their free time to build and maintain the source code. They make the code freely available in the hope that it will be of use to other developers. Please be considerate towards maintainers when raising issues or presenting pull requests. Let’s show the world that developers are civilized and selfless people.It’s the duty of the maintainer to ensure that all submissions to the project are of sufficient quality to benefit the project. Many developers have different skillsets, strengths, and weaknesses. Respect the maintainer’s decision, and do not be upset or abusive if your submission is not used.
Viability
When requesting or submitting new features, first consider whether it might be useful to others. Open source projects are used by many developers, who may have entirely different needs to your own. Think about whether or not your feature is likely to be used by other users of the project.Procedure
Before filing an issue
Replicate the problem
Attempt to replicate the problem to ensure that it wasn’t a coincidental incident.
Check existing features
Check to make sure your feature suggestion isn’t already present within the project.
Before submitting a pull request
Requirements
If the project maintainer has any additional requirements, you will find them listed here.Coding standards
Coding standards
Follow the PSR-2 Coding Standard. The easiest way to apply the conventions is to install PHP Code Sniffer.
Testing
Testing
Add tests! Your patch won’t be accepted if it doesn’t have tests.
Documentation
Documentation
Document any change in behavior. Make sure the
README.md and any other relevant documentation are kept up-to-date.Release cycle
Release cycle
Consider our release cycle. We try to follow SemVer v2.0.0. Randomly breaking public APIs is not an option.
One PR per feature
One PR per feature
If you want to do more than one thing, send multiple pull requests.
Coherent history
Coherent history
Send coherent history. Make sure each individual commit in your pull request is meaningful. If you had to make multiple intermediate commits while developing, please squash them before submitting.