mirror of
https://github.com/Llewellynvdm/conky.git
synced 2024-11-18 02:55:12 +00:00
bd5b7c87fa
* Update/fix some web stuff. Also added lefthook with some linting config. * Add note about lefthook. * Enable sort imports eslint rule. * This statement is untrue * Remove cypress video, add to gitignore * Add web CI action * Set the right dir here * Try this
39 lines
1.6 KiB
Markdown
39 lines
1.6 KiB
Markdown
# Contributing
|
|
|
|
Contributions are welcome from anyone.
|
|
|
|
## Pull Request Guidelines
|
|
|
|
When submitting PRs, please:
|
|
|
|
- Describe the changes, why they were necessary, etc
|
|
- Describe how the changes affect existing behaviour
|
|
- Describe how you tested and validated your changes
|
|
- 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
|
|
- 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.
|
|
|
|
## Git hooks
|
|
|
|
To make life easier, you can use
|
|
[lefthook](https://github.com/evilmartians/lefthook) to handle some basic
|
|
linting with a pre-commit hook, as defined in [lefthook.yml](lefthook.yml).
|
|
Follow the [installation guide for
|
|
lefhook](https://github.com/evilmartians/lefthook/blob/master/docs/install.md),
|
|
then run `lefthook install` to enable the hooks.
|
|
|
|
## Unit Testing
|
|
|
|
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.
|