search

weecology / portalr

A collection of functions to summarize the Portal Data
https://weecology.github.io/portalr/
Other
11 stars 12 forks source link

Use right location for RStudio package manager in R CMD check build #264

Closed ethanwhite closed 3 years ago

ethanwhite commented 3 years ago

Thus illustrating the value of cross-project code review :smile:

gmyenni commented 3 years ago

Ah! But the same failure is still happening. 🤔

ethanwhite commented 3 years ago

Can you link the documentation for using the RSPM environment variable the way you're doing it? I wonder if that's available in devel but not release?

gmyenni commented 3 years ago

https://github.com/r-lib/actions/blob/master/.github/workflows/check-standard.yaml

There isn't really much documentation.

It seems weird that it would be an issue with the R release version, since this was all working fine until last week.

gmyenni commented 3 years ago

Fyi, I tried updating some details to match https://github.com/tidyverse/dplyr/blob/master/.github/workflows/R-CMD-check.yaml , which passed 4 hours ago. 🤷‍♀️ I also tried reverting to 16.04, which they still use in dplyr, but that didn't help either.

gmyenni commented 3 years ago

Okay, after @henrykironde and I tried every possible thing, this does seem to legitimately be an issue with R 4.0.4. Since devel is passing, we're just going to use 4.0.3 for now, which also passes.