Skip to content

Commit 2454722

Browse files
committed
Updates to contribution instructions
1 parent 1ce5170 commit 2454722

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

CONTRIBUTING.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,7 @@ bundle exec fastlane test_ios
3434
```
3535

3636
There is also a `runAllTests` script at the repository root for convenience, which also runs `npm run lint` and `npm test`.
37+
This script requires macOS because it runs the iOS tests.
3738

3839
Notes on native unit tests and Fastlane:
3940

@@ -51,6 +52,9 @@ Notes on native unit tests and Fastlane:
5152
```
5253
- The iOS tests must be run on macOS with Xcode 8 installed.
5354
- You can also run the native unit tests in Android Studio or Xcode if you prefer, using the projects under native-tests. You must install the NPM dependencies for this repo first.
55+
- There is no need to run `pod install` or even install CocoaPods in order to run the unit tests.
56+
- If you don't have access to macOS and are not changing anything in the iOS codebase, there is no need to
57+
run the iOS unit tests. They will run in CI. If anything fails the error will be reported there.
5458

5559
### Docs
5660

0 commit comments

Comments
 (0)