👍🎉 First off, thanks for taking the time to contribute! 🎉👍
The following is a set of guidelines for contributing to DependencyGuard. These are mostly guidelines, not rules. Use your best judgment, and feel free to propose changes to this document in a pull request.
This project and everyone participating in it is governed by the Code of Conduct.
By participating, you are expected to uphold this code. Please report unacceptable behavior to mediact@github.com.
This section guides you through submitting a bug report. Following these guidelines helps maintainers and the community understand your report 📝, reproduce the behavior 💻 💻, and find related reports 🔎.
Note: If you find a Closed issue that seems like it is the same thing that you're experiencing, open a new issue and include a link to the original issue in the body of your new one.
Please use the following as guidelines to write a bug report:
A clear and concise description of what the problem is.
I'm always frustrated when [...]
A clear and concise description of what you want to happen.
I would like the following changes to be made, in order to achieve [...]
A clear and concise description of any alternative solutions or features you've considered.
I have considered using X with Y, yet [...]
Add any other context or screenshots about the feature request here.
- Do not include issue numbers in the pull request title.
- Include screenshots and animated GIFs in your pull request whenever possible.
- Follow the style guides in this document.
- Add two of the following reviewers:
- Ashoka de Wit
- Jan-Marten de Boer
- Ensure Scrutinizer CI runs successfully.
- Ensure code coverage stays the same or increases.
- Ensure code quality stays the same or improves.
The package
mediact/testing-suite
is installed, ensuring code style and standards, as well as quality.
To locally test using the testing suite, use the following:
vendor/bin/grumphp run
This will run GrumPHP using the tests set by the testing suite.
- Use the present tense ("Add feature" not "Added feature")
- Use the imperative mood ("Move cursor to..." not "Moves cursor to...")
- Limit the first line to 72 characters or less
- Reference issues and pull requests liberally after the first line