This branch is for mainline development.
This branch only works with Xcode 9.2 and supports iOS 10, and 11.
This branch is written in Swift 3.2.
Please make sure you aim your pull requests in the right direction.
We encourage you to participate in this open source project. We love Pull Requests, Bug Reports, ideas, (security) code reviews or any kind of positive contribution.
As of Sept. 16, 2016, this project requires Xcode 8.
- Install the latest Xcode developer tools from Apple.
- Install Carthage
brew update brew install carthage
- Clone the repository:
git clone https://github.com/cliqz-oss/ghostery-ios
- Pull in the project dependencies:
cd ghostery-ios sh ./bootstrap.sh
- Open
Client.xcodeproj
in Xcode. - Build the
Fennec
scheme in Xcode.
User Scripts (JavaScript injected into the WKWebView
) are compiled, concatenated and minified using webpack. User Scripts to be aggregated are placed in the following directories:
/Client
|-- /Frontend
|-- /UserContent
|-- /UserScripts
|-- /AllFrames
| |-- /AtDocumentEnd
| |-- /AtDocumentStart
|-- /MainFrame
|-- /AtDocumentEnd
|-- /AtDocumentStart
This reduces the total possible number of User Scripts down to four. The compiled output from concatenating and minifying the User Scripts placed in these folders resides in /Client/Assets
and are named accordingly:
AllFramesAtDocumentEnd.js
AllFramesAtDocumentStart.js
MainFrameAtDocumentEnd.js
MainFrameAtDocumentStart.js
To simplify the build process, these compiled files are checked-in to this repository. When adding or editing User Scripts, these files can be re-compiled with webpack
manually. This requires Node.js to be installed and all required npm
packages can be installed by running npm install
in the root directory of the project. User Scripts can be compiled by running the following npm
command in the root directory of the project:
npm run build
- Swift code should generally follow the conventions listed at https://github.com/raywenderlich/swift-style-guide.
- Exception: we use 4-space indentation instead of 2.
- New code should not contain any trailing whitespace.
- We recommend enabling both the "Automatically trim trailing whitespace" and "Including whitespace-only lines" preferences in Xcode (under Text Editing).
git rebase --whitespace=fix
can also be used to remove whitespace from your commits before issuing a pull request.
- Each commit should have a single clear purpose. If a commit contains multiple unrelated changes, those changes should be split into separate commits.
- If a commit requires another commit to build properly, those commits should be squashed.
- Follow-up commits for any review comments should be squashed. Do not include "Fixed PR comments", merge commits, or other "temporary" commits in pull requests.