-
Notifications
You must be signed in to change notification settings - Fork 28.3k
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
[SPARK-20543][SPARKR][FOLLOWUP] Don't skip tests on AppVeyor #17878
Conversation
Test build #76509 has started for PR 17878 at commit |
Test build #76510 has started for PR 17878 at commit |
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 checked the test script and it looks good to me.
Jenkins, retest this please |
Thanks, @HyukjinKwon AppVeyor looks good, waiting for Jenkins again (although, it has nothing to do with it..) |
Test build #76532 has finished for PR 17878 at commit
|
merged to master/2.2 |
## What changes were proposed in this pull request? add environment ## How was this patch tested? wait for appveyor run Author: Felix Cheung <felixcheung_m@hotmail.com> Closes #17878 from felixcheung/appveyorrcran. (cherry picked from commit 7087e01) Signed-off-by: Felix Cheung <felixcheung@apache.org>
## What changes were proposed in this pull request? add environment ## How was this patch tested? wait for appveyor run Author: Felix Cheung <felixcheung_m@hotmail.com> Closes apache#17878 from felixcheung/appveyorrcran.
What changes were proposed in this pull request?
add environment
How was this patch tested?
wait for appveyor run