[ci] migrate Windows builds from AppVeyor to GitHub Actions - #355
Merged
Merged
Conversation
AppVeyor has not run a build in years and its README badge is broken. It bootstrapped travis-tool.sh from the abandoned krlmlr/r-appveyor repo, so it is not worth repairing. Windows coverage moves to the existing GitHub Actions CI matrix, which already runs windows-latest successfully in release.yml. The ubuntu-only tidy install is already gated on matrix.os, and the remaining r-lib actions are cross-platform, so no new steps are needed. - add windows-latest / R release to the ci.yml test matrix - delete appveyor.yml and its now-dead .Rbuildignore entry - drop the broken AppVeyor badge from the README - open a development section in NEWS.md Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes #337.
AppVeyor has not run a build in years and its README badge is broken. It bootstrapped
travis-tool.shfrom the abandonedkrlmlr/r-appveyorrepo, so it is not worth repairing.Windows coverage moves to the existing GitHub Actions CI matrix. This is a low-risk move because
release.ymlalready runs awindows-latestjob successfully at R-devel — the same step sequence just needed to exist inci.yml. TheInstall Tidy Ubuntustep is already gated onmatrix.os == 'ubuntu-latest', andsetup-r/setup-pandoc/setup-tinytex/setup-r-dependencies/check-r-packageare all cross-platform, so no new steps were needed.Changes
.github/workflows/ci.yml— addwindows-latest/ Rreleaseto thetestmatrixappveyor.yml— deleted, along with its now-dead^appveyor\.yml$entry in.RbuildignoreREADME.md— drop the broken AppVeyor badgeNEWS.md— open adevelopmentsection (none existed after the 0.6.1 release bump)The 2018-era NEWS line mentioning AppVeyor is left alone as release history.
Notes for the reviewer
Whether
R CMD check --as-cranpasses onwindows-latestat R release will only be known once CI runs here. One thing to watch:release.ymlinstalls extratlmgrpackages (titling,framed,inconsolata,collection-fontsrecommended) thatci.ymldoes not. If the Windows job fails on PDF manual generation, adding those steps is the likely fix.🤖 Generated with Claude Code