Skip to content

Conversation

@carriewright11
Copy link
Contributor

@carriewright11 carriewright11 commented Nov 10, 2025

adding GA_Script.html, _bookdown.yml, and _output.yml to render-all workflow

Otherwise if people make changes to these files (and not an Rmd for example) then they need to remember to rerender the files.

This seems like something people might overlook when setting up Google Analytics for example. I have had trouble remembering to do that.

I think I need the quotes but wasn't entirely sure

adding, GA_Script.html, _bookdown.yml, and _output.yml to render-all workflow
@github-actions
Copy link
Contributor

github-actions bot commented Nov 10, 2025

OTTR Check Results

Summary

  • Spelling check: ✅ PASSED (0 errors found, threshold: 0)
  • URL check: ✅ PASSED (0 errors found, threshold: 0)

🎉 All checks passed!

Last Updated: 2025-11-10-18:36:06

@github-actions
Copy link
Contributor

github-actions bot commented Nov 10, 2025

Re-rendered previews from the latest commit:

* note not all html features will be properly displayed in the "quick preview" but it will give you a rough idea.

Updated at 2025-11-10 with changes from the latest commit 34d2339

@kweav
Copy link
Contributor

kweav commented Nov 10, 2025

@carriewright11 do you want other yml files like config_automation.yml or quarto's brand.yml too? If so, maybe just a wild card with yml?

@carriewright11
Copy link
Contributor Author

carriewright11 commented Nov 10, 2025

@carriewright11 do you want other yml files like config_automation.yml or quarto's brand.yml too? If so, maybe just a wild card with yml?

hmm yeah I think that makes sense, adding this now

Removed specific YAML files from the workflow trigger and added a wildcard for all YAML files.
Copy link
Contributor

@kweav kweav left a 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!

@carriewright11 carriewright11 merged commit 9b82f86 into main Nov 10, 2025
8 checks passed
@carriewright11 carriewright11 deleted the render_trigger branch November 10, 2025 19:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants