Skip to content
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

Transitioning docs to github actions #425

Merged
merged 5 commits into from
Jul 26, 2023
Merged

Transitioning docs to github actions #425

merged 5 commits into from
Jul 26, 2023

Conversation

aappling-usgs
Copy link
Contributor

On the GitHub site I set the pages deployment to use GitHub actions. Ran the usethis and pkgdown commands noted in the commit messages:

R CMD INSTALL
pkgdown::clean_site()
pkgdown::build_site()
usethis::use_github_action("pkgdown")
usethis::use_pkgdown_github_pages()

Will this be enough?

@aappling-usgs
Copy link
Contributor Author

aappling-usgs commented Jul 26, 2023

I tried to use pkgdown with github actions to build the vignettes, and adding the pak:pkgdepends line for unitted in the DESCRIPTION got me through the dependency installation step, but then I hit this error and got discouraged, because I know the darn thing builds locally:

-- Building articles -----------------------------------------------------------
Writing 'articles/index.html'
Reading 'vignettes/data_prep.Rmd'
Writing 'articles/data_prep.html'
Reading 'vignettes/get_started.Rmd'
-- RMarkdown error -------------------------------------------------------------

Quitting from lines 123-124 [unnamed-chunk-1] (get_started.Rmd)
--------------------------------------------------------------------------------
Error in `map()`:
ℹ In index: 2.
Caused by error in `render_rmarkdown()`:
! Failed to render RMarkdown
Caused by error:
! in callr subprocess.
Caused by error:
! can't find metabolism parameters GPP.daily, ER.daily, K600.daily
Backtrace:
     ▆
  1. ├─base::tryCatch(...)
  2. │ └─base (local) tryCatchList(expr, classes, parentenv, handlers)
  3. │   ├─base (local) tryCatchOne(...)
  4. │   │ └─base (local) doTryCatch(return(expr), name, parentenv, handler)
  5. │   └─base (local) tryCatchList(expr, names[-nh], parentenv, handlers[-nh])
  6. │     └─base (local) tryCatchOne(expr, names, parentenv, handlers[[1L]])
  7. │       └─base (local) doTryCatch(return(expr), name, parentenv, handler)
  8. ├─base::withCallingHandlers(...)
  9. ├─base::saveRDS(...)
 10. ├─base::do.call(...)
 11. ├─base (local) `<fn>`(...)
 12. └─global `<fn>`(...)
 13.   └─rmarkdown::render(...)
 14.     └─knitr::knit(knit_input, knit_output, envir = envir, quiet = quiet)
 15.       └─knitr:::process_file(text, output)
 16.         ├─base::withCallingHandlers(...)
 17.         ├─base::withCallingHandlers(...)
 18.         ├─knitr:::process_group(group)
 19.         └─knitr:::process_group.block(group)
 20.           └─knitr:::call_block(x)
 21.             └─knitr:::block_exec(params)
 22.               └─knitr:::eng_r(options)
 23.                 ├─knitr:::in_input_dir(...)
 24.                 │ └─knitr:::in_dir(input_dir(), expr)
 25.                 └─knitr (local) evaluate(...)
 26.                   └─evaluate::evaluate(...)
 27.                     └─evaluate:::evaluate_call(...)
 28.                       ├─base (local) handle(...)
 29.                       ├─base::withCallingHandlers(...)
 30.                       ├─base::withVisible(value_fun(ev$value, ev$visible))
 31.                       └─knitr (local) value_fun(ev$value, ev$visible)
 32.                         └─knitr (local) fun(x, options = options)
 33.                           ├─base::withVisible(knit_print(x, ...))
 34.                           ├─knitr::knit_print(x, ...)
 35.                           └─knitr:::knit_print.default(x, ...)
 36.                             └─evaluate (local) normal_print(x)
 37.                               ├─methods::show(x)
 38.                               └─methods::show(x)
 39.                                 ├─streamMetabolizer::get_params(...)
 40.                                 ├─streamMetabolizer:::get_params.metab_bayes(...)
 41.                                 ├─base::NextMethod()
 42.                                 └─streamMetabolizer:::get_params.metab_model(...)
Execution halted
Error: Process completed with exit code 1.

@aappling-usgs aappling-usgs merged commit 06160d3 into main Jul 26, 2023
@aappling-usgs
Copy link
Contributor Author

Nope, the pages at http://usgs-r.github.io/streamMetabolizer/ have updated content but are still not rendering correctly.
image

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.

None yet

1 participant