-
-
Notifications
You must be signed in to change notification settings - Fork 22
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
Fix additions and add GHA R CMD CHECK #90
Conversation
* test fixes and add GHA tests * test reorder * test reorder * test reorder * test reorder * Tidy
Awesome, thanks! I can confirm that I don't get the error on my computer when I run things locally but it's great to have GHA checks. Thanks for setting that up. |
Hmm for some reason the tests are hanging on the Windows run, but I can't reproduce locally. Will see if I can find a fix |
Yeah that's strange. Thanks for working on this |
It looks like it's hanging specifically at the final linking stage when creating and compiling a package, but only for Windows:
Very weird. |
@andrjohns So this is just happening on Windows with GHA but not on your local Windows system? @bgoodri Any ideas what's going wrong here? |
This looks like a GHA issue, not related to |
I wasn't able to figure out why it was hanging on the windows GHA, so have just removed for now. The new actions will test on Ubuntu & MacOS, which is still more than the previous Travis tests (just linux). |
@jgabry Would this be all good to go? Once it's out to CRAN I can submit patches to some packages for 2.26+ compatibility |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need to merge this to fix the master
branch for rstan >= 2.26
.
Duplicate of #88, but with the base branch coming from
stan-dev/rstantools
rather than my fork, to allow the Github Actions to run