-
Notifications
You must be signed in to change notification settings - Fork 0
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
Add github action to knit Rmd #2
base: main
Are you sure you want to change the base?
Conversation
stefpiatek
commented
Dec 23, 2024
•
edited
Loading
edited
- Knit and commit any changes to Rmd files in CI.
- Added renv to make my life easier
- Trivial change to Rmd to check that the action works as expected
Fallback to showing patch
Hmm frustratingly I can't install arrow locally because of a dependency issue, which means I can't test if this Rmd should knit locally or not. Failing in CI with |
…tarter-guide into stefpiatek/knit-rmd-action
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.
Good persistence :-)
@stefpiatek I see that merging seems not to be enabled until a knit check is reported. Not clear to me how we do that ? |
Yeah this is on me, I need to faff around to make an app that can give the correct token so that the action is run when a bot commits to the branch |