diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index 15eb6fac..42877cbf 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -2,12 +2,9 @@ Contributions are welcome from anyone. -To report bugs or issues, open [new issues](https://github.com/brndnmtthws/conky/issues/new). -To submit code changes, open [new pull requests](https://github.com/brndnmtthws/conky/compare). +## Pull Request Guidelines -## Guidelines - -Patches submitted in issues, email, or elsewhere may be ignored. When submitting PRs, please: +When submitting PRs, please: - Describe the changes, why they were necessary, etc - Describe how the changes affect existing behaviour @@ -15,20 +12,18 @@ Patches submitted in issues, email, or elsewhere may be ignored. When submitting - Include unit tests when appropriate - Include any relevant screenshots/evidence demonstrating that the changes work and have been tested - Any new source files should include a GPLv3 license header -- Any contributed code must be GPLv3 licensed -- Always leave the code better than you found it +- All new code must be GPLv3 licensed +- Try to leave the code better than you found it - PRs with failed checks may be ignored or closed; please make sure the build and checks pass if possible (and notify someone when the build system is not working) +_Patches submitted in issues, email, or elsewhere may be ignored._ + ## Coding Style -Code should be formatted using `clang-format`. By configuring Conky with `cmake -DCHECK_CODE_QUALITY=ON`, you will be able to run `make clang-format` to automatically format code. - -If code in your PR is not formatted according to [`.clang-format`](.clang-format), the checks will not pass. +Code should be formatted using `clang-format`. By configuring Conky with `cmake -DCHECK_CODE_QUALITY=ON`, you will be able to run `make clang-format` to automatically format code. If code in your PR is not formatted according to [`.clang-format`](.clang-format), the checks will not pass. ## Unit Testing -Unit tests are a relatively new addition to Conky, and most existing code does not have tests. Conky uses the [Catch2](https://github.com/catchorg/Catch2) unit testing framework. - -If you are adding new functions or methods, please consider adding unit tests for that code. Additionally, if you'd like to add tests for existing code, that would be a welcome contribution. +Conky uses the [Catch2](https://github.com/catchorg/Catch2) unit testing framework. If you are adding new functions or methods, please consider adding unit tests for that code. Additionally, if you'd like to add tests for existing code, that would be a welcome contribution.