| 1 |
# Contributing to Embed Privacy |
| 2 |
|
| 3 |
First, we’re happy to have you here! Thank you for taking time to contribute. :tada: :heart: |
| 4 |
|
| 5 |
## Language |
| 6 |
|
| 7 |
Even if we are based in Germany and thus speak German as well, please use English primarily in discussions on the Embed Privacy GitHub project. Use German only if you struggle to find the correct words in English. |
| 8 |
|
| 9 |
## Creating issues |
| 10 |
|
| 11 |
Before creating a new issue, please search for existing issues targeting the same problem. |
| 12 |
|
| 13 |
Please use the available issue templates for bugs and feature requests and give as much information as possible. |
| 14 |
|
| 15 |
## Adding code |
| 16 |
|
| 17 |
If you want to contribute code, please create an issue first so that we can discuss if the functionality you want to implement is intended to be a reasonable value for Embed Privacy. |
| 18 |
|
| 19 |
### Code style |
| 20 |
|
| 21 |
Basically we use the default WordPress coding style with some small adjustments. We recommend using the [](https://github.com/WordPress/WordPress-Coding-StandardsWordPress Coding Standard for PHP Code Sniffer](https://github.com/WordPress/WordPress-Coding-Standards](https://github.com/WordPress/WordPress-Coding-Standards) and exclude the following rules: |
| 22 |
|
| 23 |
```xml |
| 24 |
<rule ref="WordPress"> |
| 25 |
<exclude name="WordPress.Arrays.MultipleStatementAlignment.DoubleArrowNotAligned"/> |
| 26 |
<exclude name="WordPress.DB.SlowDBQuery.slow_db_query_meta_key"/> |
| 27 |
<exclude name="WordPress.DB.SlowDBQuery.slow_db_query_meta_value"/> |
| 28 |
<exclude name="WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedConstantFound"/> |
| 29 |
<exclude name="WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedHooknameFound"/> |
| 30 |
<exclude name="WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedVariableFound"/> |
| 31 |
<exclude name="WordPress.PHP.DisallowShortTernary.Found"/> |
| 32 |
<exclude name="WordPress.PHP.YodaConditions.NotYoda"/> |
| 33 |
<exclude name="WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedNamespaceFound"/> |
| 34 |
<exclude name="WordPress.NamingConventions.PrefixAllGlobals.ShortPrefixPassed"/> |
| 35 |
<exclude name="WordPress.WhiteSpace.DisallowInlineTabs.NonIndentTabsUsed"/> |
| 36 |
<exclude name="WordPress.WhiteSpace.PrecisionAlignment.Found"/> |
| 37 |
<exclude name="WordPress.WP.EnqueuedResourceParameters.NotInFooter"/> |
| 38 |
</rule> |
| 39 |
``` |
| 40 |
|
| 41 |
### Testing |
| 42 |
|
| 43 |
Please make sure that your code is tested in a clean environment. A huge plus will be to know the code is tested in both a single and a multisite installation. |
| 44 |
|
| 45 |
### Commit message |
| 46 |
|
| 47 |
We use [](https://gitmoji.devgitmoji](https://gitmoji.dev](https://gitmoji.dev) to get a decent overview over what’s the purpose of a single commit. Thus, you should use them as well. |
| 48 |
|
