Skip to content

Disable multienrichjam install code, exclude it from workshop.R - #39

Merged
hosseinvk merged 1 commit into
mainfrom
hosseinvk/disable-multienrichjam-install
Aug 10, 2026
Merged

Disable multienrichjam install code, exclude it from workshop.R#39
hosseinvk merged 1 commit into
mainfrom
hosseinvk/disable-multienrichjam-install

Conversation

@hosseinvk

Copy link
Copy Markdown
Collaborator

Summary

  • Comments out the install.packages("remotes")/library(remotes)/remotes::install_github(...)/library(multienrichjam)/?enrichDF2enrichResult lines in the optional multienrichjam install chunk in 10-clusterprofiler.Rmd, so they can't be run accidentally (chunk already had eval = FALSE, but that only affects knitting, not manual execution).
  • Wraps the whole chunk in <!--omit-begin-->/<!--omit-end--> so it's excluded entirely from the generated workshop.R, while remaining visible (as before) in the rendered book chapter.

Test plan

  • Full bookdown::render_book() completes cleanly; confirmed the chunk still renders in docs/clusterprofiler-and-enrichplot.html.
  • Regenerated workshop.R and confirmed none of the install/library/help lines appear anywhere in it.
  • CI bookdown-render job to be confirmed green before merge.

Comments out the install.packages/remotes/library/help lines in the
optional multienrichjam install chunk so they can't be accidentally
run, and wraps the whole chunk in <!--omit-begin/end--> so it's
excluded from the generated workshop.R entirely while remaining
visible in the rendered book chapter.
@hosseinvk
hosseinvk merged commit 8d466ae into main Aug 10, 2026
1 of 2 checks passed
@hosseinvk
hosseinvk deleted the hosseinvk/disable-multienrichjam-install branch August 10, 2026 12:41
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.

1 participant