Download on the App Store.
This branch is for mainline development that will ship in the next release.
This branch currently supports iOS 11, and is written in Swift 4.
Please make sure you aim your pull requests in the right direction.
For bug fixes and features for the upcoming release, please see the associated GitHub milestones (e.g. 2.1.3).
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.
- Discord:
#ios
channel for general conversing.#developers-ios
channel for development discussion.
- Bugs: File a new bug • Existing bugs
Want to contribute but don't know where to start? Here is a list of Good First Issues.
- Install the latest Xcode developer tools from Apple.
- Install Carthage:
brew update brew install carthage
- Clone the repository:
git clone https://github.com/brave/brave-ios.git
- Pull in the project dependencies:
cd brave-ios sh ./bootstrap.sh
- Open
Client.xcodeproj
in Xcode. - Build the
Fennec
scheme in Xcode.
- All pull requests must be associated with a specific GitHub issue.
- If a bug corresponding to the fix does not yet exist, please file it.
- Please use the following formats in your PR titles:
Fix/Ref #<issueId>: <description>.
Examples:
Fix #102: Added Face ID usage description to plist.
Ref #102: Fixed type on Face ID usage description.
- Add any additional information regarding the PR in the description.
- In the unlikely and rare situation that a PR fixing multiple, related issues separate issue numbers with a comma:
Fix #159, Fix #160: Removed whitepsace for + button on right-side panel.
- PRs will be squashed and merged, so it is important to keep PRs focused on specific tasks.
- 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 the "Automatically trim trailing whitespace" and keeping "Including whitespace-only lines" deselected in Xcode (under Text Editing).
- 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.
In most cases Pull Request commits will remain intact with a merge commit on the targeted branch.
- After running the bootstrap.sh script in the setup instructions navigate to:
Client/Configuration/Local/DevTeam.xcconfig
- Add your Apple Team ID in this file:
LOCAL_DEVELOPMENT_TEAM = KL8N8XSYF4
Team IDs look identical to provisioning profile UUIDs, so make sure this is the correct one.
The entire Local
directory is included in the .gitignore
, so these changes are not tracked by source control. This allows code signing without making tracked changes. Updating this file will only sign the Fennec
target for local builds.
The easiest known way to find your team ID is to log into your Apple Developer account. After logging in, the team ID is currently shown at the end of the URL:
https://developer.apple.com/account/<TEAM ID>
Use this string literal in the above, DevTeam.xcconfig
file to code sign