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

[ci] switch from MiKTeX to tinytex on Windows R jobs (fixes #5600) #5602

Merged
merged 1 commit into from
Nov 25, 2022

Conversation

jameslamb
Copy link
Collaborator

@jameslamb jameslamb commented Nov 24, 2022

Fixes #5600.

R CI jobs require a LaTeX distribution to build the package documentation. This PR proposes switching from MiKTeX to {tinytex} for Windows builds.

Motivation

The use of MiKTeX has been the source of several multi-day outages in LightGBM's CI over the last few years (and most recently, #5600):

Several of those have had the resolution "wait a few days until changes in CTAN mirrors or other upstream sources are fixed". That's very disruptive to our development here, as it means either skipping R package CI jobs or not merging anything for multiple days.

I'm hoping that because {tinytex} is lighter-weight, it'll be more stable.

I also believe using the GitHub Action https://github.com/r-lib/actions/tree/v2-branch/setup-tinytex will remove the maintenance burden of keeping up with MiKTeX and other updates upstream, since that action is actively maintained.

For more, see "Why TinyTex?" in the {tinytex} docs.

@jameslamb
Copy link
Collaborator Author

@guolinke the next time you have a few minutes, can you please review this PR? This will unblock CI, which will unblock #5396, which will prevent CI from breaking starting 8 days from now when Azure removes the macos-10.15 environment.

I'm sorry to keep @-ing you, but there is a lot that needs to get done here, there hasn't been much activity recently from other maintainers, and I am trying to get the project into a good state before some planned travel starting next week.

@guolinke
Copy link
Collaborator

@jameslamb no problems, please feel free to @ me when you need any helps.

@jameslamb
Copy link
Collaborator Author

Thank you so much! I'll merge this, merge master into #5396, and then merge #5396 once it builds successfully.

@github-actions
Copy link

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.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 19, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[ci] [R-package] Miktex installation failing: "Timeout was reached"
2 participants