-
Notifications
You must be signed in to change notification settings - Fork 31
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
Should we expand the cicd testing via rhub (and re-render readme)? #202
Comments
Yes, i think that would be a good idea.
Is there anyway we could set up something to push to develop to
automatically build documents including manual?
…On Mon, May 31, 2021 at 10:04 PM James Black ***@***.***> wrote:
https://r-hub.github.io/rhub/articles/rhub.html
above lets you test across many more platforms- should we look to more
than the current few platforms?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#202>, or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AEVWOL6KHNHHYMEGE4TCTGTTQPTUNANCNFSM453MXOXQ>
.
|
Just completing the Slack message - the CICD process only writes to the deployed documentation on the pages side, it doesn't write to the base branch. Master is locked, e.g. only way in is via a PR, so we could potentially do it - but then it's more a philosophical issue of saving built artefacts to the main branch. |
We test across
I guess redhat and cent0S are used (RHEL in particular) but missing. |
will try adding this: https://github.com/r-lib/actions/tree/master/examples#render-rmarkdown for
|
ok - so now if the readme is edited, it will auto rebuild (on any branch) @bailliem. Is this what you meant? So the 'plots' won't update on the README if nothing has changed in the readme, but they will update on the docs site. But if someone makes any changes to README.rmd, then it will make sure the version on github has the rebuilt readme.md. |
maybe this one works.. https://github.com/openpharma/visR/runs/3054578553?check_suite_focus=true |
ok - I'll just share the index of all (forgot to add a package to the gh-action so 2nd one also failed) |
https://r-hub.github.io/rhub/articles/rhub.html
above lets you test across many more platforms- should we look to more than the current few platforms?
The text was updated successfully, but these errors were encountered: