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

Decrease R mininum version to 4.0.0 #404

Closed
wants to merge 2 commits into from
Closed

Decrease R mininum version to 4.0.0 #404

wants to merge 2 commits into from

Conversation

joshwlambert
Copy link
Member

@joshwlambert joshwlambert commented Oct 22, 2024

This PR changes the minimum R version required by {epiparameter} to 4.0.0, after it was incremented to 4.1.0 in PR #384 (due to mitchelloharawild/distributional#128).

This PR also adds a R-CMD-check CI run for R 4.0 to check that the R package passes the check on the version stated in the DESCRIPTION.

This PR initially checked to see if the minimum R version could be reverted to 3.6.0 as it was before #384, however, due to {epireview} depending on {flextable} which in turn depends on {gdtools} which requires R 4.0, then this is required for the R-CMD-check workflow to install {epireview}.

Usage of the base pipe (|>) has been replaced in a vignette by the pipe from {magrittr} (%>%), with {magrittr} added as a suggested dependency.

The note in the design_principles.Rmd vignette on breaking the Epiverse R version policy has been removed as this will not longer be the case.

@joshwlambert joshwlambert added CI pkg infrastructure General update tasks relating to the package and not functionality labels Oct 22, 2024
@joshwlambert joshwlambert changed the title Decrease R mininum version to 3.6.0 Decrease R mininum version to 4.0.0 Oct 22, 2024
@joshwlambert
Copy link
Member Author

This PR fails the R-CMD-check CI for both R v3.6.0 and v4.0.0. Due to {epireview} depending on {gdtools} which requires R 4.0 and then {distributional} which released v0.5.0 using the base pipe (|>). The base pipe has since been removed from {distributional} but this has not yet been released to CRAN.

Therefore, I will close this PR without merging, and retain the minimum R version of 4.1.0 for {epiparameter} and add a explicit R-CMD-check CI for R 4.1 on a separate PR.

The alternative would be to wait to for a new {distributional} release to enable the minimum R version to be 4.0, but this seems like a marginal gain and would be a blocker to releasing {epiparameter}.

@joshwlambert
Copy link
Member Author

Closing, superseded by #405.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI pkg infrastructure General update tasks relating to the package and not functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant