blob: cd28b16dd0ba0c4cdd8aa01686520f4aa428ab1f (
plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
|
# Working with Privacy Badger's code
To make changes to Privacy Badger, you have to first load the extension from a source code checkout.
## Install from source
To install Privacy Badger from source in Chrome, visit `chrome://extensions`, enable "Developer mode", click "Load unpacked" and select the [`src`](src/) subdirectory inside your copy of the Privacy Badger source code.
In Firefox, visit `about:debugging`, click "Load Temporary Add-on" and select the [`src/manifest.json`](src/manifest.json) file. Note that this only installs the extension temporarily; it will be removed when you close Firefox.
To install Privacy Badger from source in Firefox for Android, please see [Mozilla's guide to developing extensions for Firefox for Android](https://extensionworkshop.com/documentation/develop/developing-extensions-for-firefox-for-android/) and [`web-ext` documentation](https://extensionworkshop.com/documentation/develop/getting-started-with-web-ext/#testing-in-firefox-for-android).
## Send a pull request
Before submitting a pull request (PR), please review the sections below.
### Style guide
All JavaScript code going forward should use the following naming conventions:
- Objects and their properties should be Java or camelCase style.
- Primitive types should be Python or snake_case style.
- Constants should be ALL_CAPS.
Examples:
```javascript
const TRACKER_ENTROPY_THRESHOLD = 33;
let tab_id = details.tabId;
window.badger.getTrackerCount(tab_id);
```
### Catch errors early with static code analysis
First, install the exact expected version of [ESLint](https://eslint.org) by running `npm install` in your Privacy Badger source code checkout directory. You should then be able to produce a lint report by running `make lint` in the same directory.
You can review our set of ESLint rules in [`.eslintrc.yml`](/.eslintrc.yml). Files we want ESLint to ignore are specified in [`.eslintignore`](/.eslintignore).
### Commit messages
Please review the suggestions in this excellent [guide to writing commit messages](https://chris.beams.io/posts/git-commit/).
|
