search

carpentries-incubator / bioc-project

Introduction to the Bioconductor project
https://carpentries-incubator.github.io/bioc-project
Other
18 stars 15 forks source link

update package cache #70

Closed zkamvar closed 1 year ago

github-actions[bot] commented 1 year ago

Thank you!

Thank you for your pull request :smiley:

:robot: This automated message can help you check the rendered files in your submission for clarity. If you have any questions, please feel free to open an issue in {sandpaper}.

If you have files that automatically render output (e.g. R Markdown), then you should check for the following:

Rendered Changes

:mag: Inspect the changes: https://github.com/carpentries-incubator/bioc-project/compare/md-outputs..md-outputs-PR-70

The following changes were observed in the rendered markdown documents:

 01-setup.md                        |  4 +-
 02-introduction-to-bioconductor.md | 16 ++++----
 03-installing-bioconductor.md      | 80 ++++++++------------------------------
 04-getting-help.md                 |  6 +--
 05-s4.md                           | 24 +++++++-----
 06-biological-sequences.md         | 14 +++----
 07-genomic-ranges.md               | 12 +++---
 md5sum.txt                         | 34 ++++++++--------
 8 files changed, 75 insertions(+), 115 deletions(-)
What does this mean? If you have source files that require output and figures to be generated (e.g. R Markdown), then it is important to make sure the generated figures and output are reproducible. This output provides a way for you to inspect the output in a diff-friendly manner so that it's easy to see the changes that occur due to new software versions or randomisation.

:stopwatch: Updated at 2023-05-30 16:59:34 +0000

zkamvar commented 1 year ago

The package cache was broken due to not yet figuring out https://github.com/carpentries/sandpaper/issues/453

To get this to work, I ran:

sandpaper::manage_deps() # got an error here
sandpaper::update_cache()
zkamvar commented 1 year ago

This is just a reminder that in order to get the builds for this working again, this PR and #70 need to be merged.

kevinrue commented 1 year ago

thanks, so sorry, I'm tied in teaching and forgot about this, will look at it asap