Brave for iOS 🦁
Download on the App Store.
This branch (development)
This branch is for mainline development that will ship in the next release.
This branch currently supports iOS 15+, and is written in Swift 5.
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).
Getting involved
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.
- Development discussion: 'Contributing-ios' Community Forums:
- 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.
Building the code
- Install the latest Xcode developer tools from Apple. (Xcode 14.0 and up required).
- Install Xcode Command Line Tools
xcode-select --install
- Make sure
npmis installed,nodeversion 16 is recommended - Install SwiftLint (0.50.0 or higher):
brew update brew install swiftlint
- Clone the repository:
git clone https://github.com/brave/brave-ios.git
- Pull in the project dependencies:
cd brave-ios sh ./bootstrap.sh --ci - Add a symlink to
npm(M1 Macs)sudo ln -s $(which npm) /usr/local/bin/npm sudo ln -s $(which node) /usr/local/bin/node
- Open
App/Client.xcodeprojin Xcode. - Build the
Debugscheme in Xcode.
Working with BraveCore
Many features in iOS (sync, ads, wallet, etc.) are powered by shared code in brave-core. Instructions on building and updating this code can be found here
Contributor guidelines
Creating a pull request
- 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 style
- Swift code should generally follow the conventions listed at https://github.com/raywenderlich/swift-style-guide.
Whitespace
- 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).
Commits
- 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.
Code Signing
- After running the bootstrap.sh script in the setup instructions navigate to:
App/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 Debug target for local builds.
Finding Team IDs
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
Attribution
This repository is a fork of Firefox iOS Browser