Skip to content
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

something wrong with ci #1221

Closed
afc163 opened this issue Apr 1, 2018 · 8 comments
Closed

something wrong with ci #1221

afc163 opened this issue Apr 1, 2018 · 8 comments

Comments

@afc163
Copy link
Member

afc163 commented Apr 1, 2018

image

用例报错了但是结果是成功的。

https://ci.appveyor.com/project/afc163/ant-design-pro/build/1.0.1152

@afc163 afc163 changed the title something wrong with appveyor ci something wrong with ci Apr 1, 2018
@afc163
Copy link
Member Author

afc163 commented Apr 1, 2018

@chenshuai2144
Copy link
Collaborator

#1133

和这个一样吧,别名的问题

@afc163
Copy link
Member Author

afc163 commented Apr 1, 2018

这个问题是 test 失败了,ci 还提示成功。

@chillyistkult
Copy link

chillyistkult commented Apr 12, 2018

@afc163 @chenshuai2144 Even if you fix the alias issue, the tests will still not work on travis or circleci. Because Jest always returns with exit code 0! Build providers will not mark the test as failed.

jestjs/jest#2470

I guess you have to upgrade jest dependencies in roadhog / umi.

@chenshuai2144
Copy link
Collaborator

@chillyistkult thank you!

@chenshuai2144 chenshuai2144 mentioned this issue May 2, 2018
@afc163
Copy link
Member Author

afc163 commented May 2, 2018

I guess you have to upgrade jest dependencies in roadhog / umi.

@sorrycc

@afc163
Copy link
Member Author

afc163 commented May 9, 2018

afc163/umi-dva-user-dashboard#1

umi test 有些问题。

@chenshuai2144
Copy link
Collaborator

#1414

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants