-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
[ci] use GitHub Actions for R CI jobs (fixes #2353) #3116
Conversation
@guolinke I think you need to enable GitHub Actions in this repository for them to build |
@jameslamb Can you please create a branch under this repo and merge you fork into it? Then we will be able to see the result! |
ohhhhh I didn't realize that was necessary! Ok I'll create a LightGBM branch and open a new PR...so we can see the status as things are changed after reviews |
moved to #3119 |
This pull request has been automatically locked since there has not been any recent activity since it was closed. To start a new related discussion, open a new issue at https://github.com/microsoft/LightGBM/issues including a reference to this. |
This PR proposes adding GitHub Actions CI and moving all R jobs to it.
GitHub Actions gives you 20 concurrent jobs for free and access to Mac, Linux, and Windows environments.
See #2353 and discussion in this thread: #3065 (comment)
This PR should improve CI times (since AppVeyor runs tasks sequentially and now those tasks will be run concurrently), and should give enough extra CI capacity to test more installation paths for the R package(like R 3.6.x AND R 4.0.0x on #3065 )
Summary of Changes
lint
job from Travis to GitHub Actions.ci/download-miktex.R
, to makemiktexsetup.zip
downloads more reliableNotes for reviewers
r-package
job, since it gives us access to Visual Studio 2017. GitHub Actions only has Visual Studio 2019 (reference).ci/download-miktex.R
because maybe 1 out of every 8 builds hit this issue: [ci] peg MiKTeX mirror #3113 (comment)stderr
causes the build to fail, even if the command itself succeeds. I tracked that down to this section of the docs, which state:Other references