Skip to content

Conversation

@kweav
Copy link
Contributor

@kweav kweav commented May 9, 2025

Adding a step to .github/workflows/test.yml and a directory test-rmd_web to test "rmd_web" rendering. Within that directory I've added

  • _site.yml
  • config_automation.yml with render-website option set to "rmd_web" and the render-leanpub and render-coursera options set to no (can I remove these options without issue?)
  • normal files within resources/
  • an index.Rmd file
  • a styles.css

I haven't included any of the .github/workflows subdirectory files within the test-rmd_web directory yet. Seeing if they are needed.

@github-actions
Copy link

github-actions bot commented May 9, 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-05-09 with changes from the latest commit a67ff65

@kweav
Copy link
Contributor Author

kweav commented May 9, 2025

Encountering an error in the Testing ottr-preview action here:

Run if [ 'test-rmd_web' != '.' ]; then
  if [ 'test-rmd_web' != '.' ]; then
    zip -r /home/runner/work/ottr-preview/ottr-preview/test-rmd_web/website-preview.zip \
    /home/runner/work/ottr-preview/ottr-preview/test-rmd_web/docs/*
  else
    zip -r /home/runner/work/ottr-preview/ottr-preview/website-preview.zip \
    /home/runner/work/ottr-preview/ottr-preview/docs/* -r
  fi
  shell: /usr/bin/bash --noprofile --norc -e -o pipefail {0}
	zip warning: name not matched: /home/runner/work/ottr-preview/ottr-preview/test-rmd_web/docs/*

zip error: Nothing to do! (try: zip -r /home/runner/work/ottr-preview/ottr-preview/test-rmd_web/website-preview.zip . -i /home/runner/work/ottr-preview/ottr-preview/test-rmd_web/docs/*)

Looks like rendering was successful because of the note Rendering status success above this chunk in the logs. Maybe I need to add a docs folder?

@kweav
Copy link
Contributor Author

kweav commented May 9, 2025

However, both of the other pre-existing test directories output rendered to docs and don't have docs folders

@kweav
Copy link
Contributor Author

kweav commented May 9, 2025

I think I misunderstood the rendering success message because there was an error in the rendering of the index.Rmd file

Error in yaml::yaml.load(..., eval.expr = TRUE) :
Scanner error: mapping values are not allowed in this context at line 2, column 7

Attempting a fix and retrying

@kweav
Copy link
Contributor Author

kweav commented May 9, 2025

It worked and didn't need the subdirectory workflow files!
image

@kweav
Copy link
Contributor Author

kweav commented May 9, 2025

Going to test what happens if I don't have the explicit no's on render-leanpub and render-coursera

@kweav
Copy link
Contributor Author

kweav commented May 9, 2025

Also works with no error. Thinking about what this means for changes to the OTTR_Template_Website repo and its sync from the OTTR_Template

kweav added 2 commits June 13, 2025 16:12
update to point to this branch
bring cansavvy/no-docx changes over here
@kweav kweav changed the title try without workflow yml files adding an rmd_web test and just testing things out Jun 13, 2025
kweav added 5 commits June 16, 2025 14:02
bring changes from kweav-suppress-docx branch over here
trigger PR stuff hopefully
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.

2 participants