Skip to content
This repository has been archived by the owner on Jun 3, 2021. It is now read-only.

Upload IOS Travis CI #10

Merged
merged 31 commits into from
Jul 19, 2019
Merged

Upload IOS Travis CI #10

merged 31 commits into from
Jul 19, 2019

Conversation

wrmswindmill
Copy link
Contributor

@wrmswindmill wrmswindmill commented Jul 18, 2019

if WeexDemo build success and pass all the test case in WeexUITestDemo, then it will be success.
but there are not any test cases at all(only the test case template now!)

the following is the shell command:

xcodebuild  -workspace WeexDemo.xcworkspace test -scheme WeexDemo CODE_SIGN_IDENTITY="" CODE_SIGNING_REQUIRED=NO -destination "platform=iOS Simulator,name=iPhone 6"

.travis.yml Outdated Show resolved Hide resolved
dangerfile-ios.js Outdated Show resolved Hide resolved
@wrmswindmill wrmswindmill reopened this Jul 18, 2019
@wrmswindmill
Copy link
Contributor Author

wrmswindmill commented Jul 18, 2019

I'm sorry to test so much times online, because my local programming env is broken.(I am fixing it) and in my Travis CI ,it build success(but actually it should be failed)

@wrmswindmill
Copy link
Contributor Author

wrmswindmill commented Jul 18, 2019

Error log:

so 'base/third_party/icu/icu_utf.h' file not found

this error caused by the pr in weex-sdk :apache/incubator-weex#2693
it change the file WeexSDK.podspec at root directory,

exclude:
'weex_core/Source/base/message_loop',
'weex_core/Source/base/third_party',
 'weex_core/Source/base/thread',

so base/third_party/icu/icu_utf.h not found.

Copy link
Member

@wqyfavor wqyfavor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some xcuserdata file should not be committed.

Copy link
Member

@wqyfavor wqyfavor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK

@wqyfavor wqyfavor merged commit 91ea38c into apache:master Jul 19, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants