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

WIP: [ci] [R-package] add a CI job to run R reverse dependency checks #6734

Draft
wants to merge 34 commits into
base: master
Choose a base branch
from

Conversation

jameslamb
Copy link
Collaborator

@jameslamb jameslamb commented Dec 2, 2024

Related to #6714

This proposes adding an optional CI job that runs a development build of the R package against all of the packages on CRAN that depend on it.

Hoping that this + manually searching https://github.com/cran will help us understand whether user-facing breaking changes will break other packages on CRAN.

Notes for Reviewers

Just opening this to test. Will fill out a better. description before putting it up for review.

ref: r-lib/actions#332 (comment)

Why not run this on PRs?

It's slow and expensive (it downloads and installs 130+ R packages). In my experience, more than half of PRs here do not even touch C++ or R code, and so shouldn't impact the result of this check.

This task of checking reverse dependencies is so expensive that at least as of 2021, Posit (formerly RStudio) was running a dedicated internal service just to do that (r-lib/actions#332 (comment)).

It's not TOO bad right now given the small number of packages on CRAN that depend on {lightgbm}... if that even gets much larger and makes this too expensive to run in CI, that'll be a good problem to have 😊

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

Successfully merging this pull request may close these issues.

1 participant