Skip to content

Commit

Permalink
revert all the way back to \dontrun and explain in the CRAN comments
Browse files Browse the repository at this point in the history
  • Loading branch information
ChrisRackauckas committed Aug 4, 2021
1 parent 5ad0d82 commit 4d4a3de
Showing 1 changed file with 7 additions and 7 deletions.
14 changes: 7 additions & 7 deletions cran-comments.md
Original file line number Diff line number Diff line change
Expand Up @@ -11,13 +11,13 @@ Proper use of this package requires a Julia and DifferentialEquations.jl install
This is noted in the installation guide. The Github Actions tests show that on
the major operating systems, if this is installed, then the package will successfully
pass its tests. However, since these softwares are not available on all of the CRAN
computers the tests fail as expected there, and are thus skipped.

Previously, the package was archived on CRAN due to accidentally installing
Julia on the CRAN computers. Given the comments from the Twitter thread
https://twitter.com/ChrisRackauckas/status/1405518810958991361, the community
suggested the fix of using \dontrun instead of \donttest to the offending examples,
which is the fix that has been implemented here.
computers the tests fail as expected there, and are thus skipped. Uwe had suggested
updating the CRAN Julia installation, but this will still require a
DifferentialEquations.jl installation, which itself will pull in other binaries,
will be something we will want to be frequently updated, etc. which I can
foresee leading to its own set of issues. Thus we believe the best approach is
to simply \dontrun on CRAN and use our own CI for handling the various
operating systems and binary installations.

## R CMD check results

Expand Down

0 comments on commit 4d4a3de

Please sign in to comment.