-
Notifications
You must be signed in to change notification settings - Fork 26
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Run unit tests for iOS code on CI server #242
base: 156-setup-travis-ci-to-have-a-build-jobs
Are you sure you want to change the base?
Run unit tests for iOS code on CI server #242
Conversation
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. |
131f9b0
to
22fa6a1
Compare
e7d5b47
to
04eb44f
Compare
8b5fe2c
to
04eb44f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few concerns
@@ -55,6 +55,11 @@ jobs: | |||
- IOS_BUILD_TYPE='Debug' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
let's rename the job to something like Debug / Tests / Simulator / Xcode 11.2
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What about
iOS / Debug (Tests) / Simulator / Xcode 11.2
instead of
iOS / Debug / Simulator / Xcode 11.2
?
@@ -55,6 +55,11 @@ jobs: | |||
- IOS_BUILD_TYPE='Debug' | |||
- IOS_PLATFORM_TYPE='iphonesimulator' | |||
- DST='platform=iOS Simulator,OS=13.2.2,name=iPhone 11' | |||
script: | |||
- scripts/ci/build.sh --build MapboxVision |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the tests will build MapboxVision
as a dependency, there is no need to waste time on a standalone build
b4a676f
to
2982213
Compare
568fe3b
to
a39cb36
Compare
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. |
NOTE: branch has been branched off from the
156-setup-travis-ci-to-have-a-build-jobs
branch. It's more convenient to see the diff.That means we'd prefer to merge #240 first and then I will change the base branch.
Here's the test results: https://travis-ci.com/github/mapbox/mapbox-vision-ios/builds/159633808
Checks:
dev
branchLinked issues:
Resolves #109