-
Notifications
You must be signed in to change notification settings - Fork 1
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
deploy failure dev shiny #213
Comments
fixed the inline comment, but have further issues regarding CRAN mirrors. Haven't been able to solve. Deploy error Error in contrib.url(repos, type) : |
Sounds like we have "install.packages" somewhere in the code - that should not anywhere |
Ignore that comment - clearly the issue is in the action, not our code |
Currently, we are setting up R in two ways -
AND
I believe we still use the second version even after your change (which allows for much faster package install) - but should not set up both. I will try to track down how we can change the version in the r-ci version, and see if that fixes the CRAN issue (that version uses an inbuilt CRAN, so we should not change it) |
I asked the r2u author whether we can specify the R version: eddelbuettel/r2u#74 - that approach is faster and more robust, so I would not want to rebuild the action for now. R 4.2.2 was released 8 days ago, and shinyapps.io usually wants to support new versions within a couple of days - so the problem will go away temporarily very soon anyway, but we should obviously know how to fix this when it recurrs. |
It cannot be done with r2u, and I would not want to fix the R version in our automated deploy script as that would mean updating that regularly. Instead, I added a manual deploy script that we can use when shinyapps.io and the latest R diverge. To run that, go to Actions and select document-and-deploy-manual, click on |
@TNRiley hope that makes sense - let me know if you run into further issues / have better ideas. |
I realized that the performance catches up when the action is run more than once (due to caching) and that it is better to have only one version - so I switched to this, and we now always deploy to shinyapps.io with the previous R version. That should be robust and easier to maintain than two actions. @TNRiley so no manual deploy is needed. |
sounds good. much of this deployment stuff is over my head as I've not yet played around with it. Appreciate you working on this and resolving it. I see the latest now has the fixes I had made. |
Inline comment causing deployment failure
The text was updated successfully, but these errors were encountered: