Skip to content

Conversation

@topepo
Copy link
Member

@topepo topepo commented Mar 17, 2019

This initially started as a simple change for #115 but....

  • r-devel has changed the rules on what package code can be written in a unit test (even if it is being skipped). As a potentially temporary change, I've added a bunch of packages to Suggests...

  • ... and that triggered travis hell initially related to rstan Cpp requirements.

  • Then I figured out that tests were failing because r-devel made changes to the random number generator in sample. Test cases were updated.

```
* installing *source* package ‘devtools’ ...
** package ‘devtools’ successfully unpacked and MD5 sums checked
** R
** inst
** byte-compile and prepare package for lazy loading
Error in loadNamespace(j <- i[[1L]], c(lib.loc, .libPaths()), versionCheck = vI[[j]]) :
  there is no package called ‘magrittr’
ERROR: lazy loading failed for package ‘devtools’
* removing ‘/home/travis/R/Library/devtools’
* restoring previous ‘/home/travis/R/Library/devtools’
The downloaded source packages are in
	‘/tmp/RtmpUhu1j4/downloaded_packages’
Warning message:
In install.packages(c("devtools")) :
  installation of package ‘devtools’ had non-zero exit status
```
@topepo topepo merged commit be85643 into master Mar 17, 2019
@github-actions
Copy link
Contributor

github-actions bot commented Mar 8, 2021

This pull request has been automatically locked. If you believe you have found a related problem, please file a new issue (with a reprex: https://reprex.tidyverse.org) and link to this issue.

@github-actions github-actions bot locked and limited conversation to collaborators Mar 8, 2021
@hfrick hfrick deleted the no-factor-fail branch September 17, 2021 14:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants