-
Notifications
You must be signed in to change notification settings - Fork 336
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
updated gha instructions/workflows for quarto #2757
Conversation
Some of the actions are failing for some reason, I think one that needs resolved in r-lib/actions? |
@gaborcsardi any idea why we're missing quarto here? |
You might try setting |
given tinytex was being installed by the quarto action that this PR removes, we may need to add: - uses: r-lib/actions/setup-tinytex@v2 |
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.
looks good to me. pkgdown needs to install tinytex to build one of the test vignettes, but most other packages shouldn't need it.
Perfect, thank you! |
Closes #2743
Updates quarto vignette and gha workflow files now that
setup-r-dependencies
automatically installs quarto as required.