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

use handouts #97

Merged
merged 5 commits into from
May 10, 2023
Merged

use handouts #97

merged 5 commits into from
May 10, 2023

Conversation

lgatto
Copy link
Collaborator

@lgatto lgatto commented May 9, 2023

No description provided.

@github-actions
Copy link

github-actions bot commented May 9, 2023

⚠️ WARNING ⚠️

This pull request contains a mix of workflow files and regular files. This could be malicious. No preview will be created.

regular files:

  • episodes/20-r-rstudio.Rmd
  • episodes/23-starting-with-r.Rmd
  • episodes/25-starting-with-data.Rmd
  • episodes/30-dplyr.Rmd
  • episodes/40-visualization.Rmd

workflow files:

  • .github/workflows/sandpaper-main.yaml

@lgatto
Copy link
Collaborator Author

lgatto commented May 9, 2023

Testing the workflow to generate handouts (see #96)

@lgatto
Copy link
Collaborator Author

lgatto commented May 9, 2023

Adding another commenet to fix check test

Remove 

          options(sandpaper.handout = TRUE)

to avoid warning
@lgatto
Copy link
Collaborator Author

lgatto commented May 9, 2023

@zkamvar - the following PR only changes/adds some purl=TRUE options to code chunks (so that later, I can add handouts).

The checks fail at the moment, and I'm not sure why, but it could be related to caching that was locked under R 4.2, while now the action uses 4.3. How can I deleted/replace the cache (if that is the issue)?

@zkamvar
Copy link
Contributor

zkamvar commented May 9, 2023

I'm running the update cache workflow to see if that can bump your versions: https://github.com/carpentries-incubator/bioc-intro/actions/runs/4930588218. I apologize, the bot wasn't activated for this repo.

Because this is a bioconductor lesson, I'm not terribly familiar about what happens around the boundaries between Bioc releases and CRAN releases. I'm trying to build it on my computer to see what happens and I will report back.

@zkamvar
Copy link
Contributor

zkamvar commented May 9, 2023

Let's hope that #98 works 🤞🏼

After that, I'm going to take a look at the renv.lock file and see if there is anything that I can do to avoid this situation in the future.

github-actions bot pushed a commit that referenced this pull request May 10, 2023
use options(sandpaper.handout = TRUE)
@lgatto lgatto merged commit 5f57bc7 into main May 10, 2023
github-actions bot pushed a commit that referenced this pull request May 10, 2023
Auto-generated via {sandpaper}
Source  : 5f57bc7
Branch  : main
Author  : Laurent Gatto <[email protected]>
Time    : 2023-05-10 08:51:20 +0000
Message : Merge pull request #97 from carpentries-incubator/lgmain

use handouts
github-actions bot pushed a commit that referenced this pull request May 10, 2023
Auto-generated via {sandpaper}
Source  : 8fed86b
Branch  : md-outputs
Author  : GitHub Actions <[email protected]>
Time    : 2023-05-10 08:53:44 +0000
Message : markdown source builds

Auto-generated via {sandpaper}
Source  : 5f57bc7
Branch  : main
Author  : Laurent Gatto <[email protected]>
Time    : 2023-05-10 08:51:20 +0000
Message : Merge pull request #97 from carpentries-incubator/lgmain

use handouts
github-actions bot pushed a commit that referenced this pull request May 16, 2023
Auto-generated via {sandpaper}
Source  : 8fed86b
Branch  : md-outputs
Author  : GitHub Actions <[email protected]>
Time    : 2023-05-10 08:53:44 +0000
Message : markdown source builds

Auto-generated via {sandpaper}
Source  : 5f57bc7
Branch  : main
Author  : Laurent Gatto <[email protected]>
Time    : 2023-05-10 08:51:20 +0000
Message : Merge pull request #97 from carpentries-incubator/lgmain

use handouts
github-actions bot pushed a commit that referenced this pull request May 30, 2023
Auto-generated via {sandpaper}
Source  : 8fed86b
Branch  : md-outputs
Author  : GitHub Actions <[email protected]>
Time    : 2023-05-10 08:53:44 +0000
Message : markdown source builds

Auto-generated via {sandpaper}
Source  : 5f57bc7
Branch  : main
Author  : Laurent Gatto <[email protected]>
Time    : 2023-05-10 08:51:20 +0000
Message : Merge pull request #97 from carpentries-incubator/lgmain

use handouts
github-actions bot pushed a commit that referenced this pull request Jun 6, 2023
Auto-generated via {sandpaper}
Source  : 8fed86b
Branch  : md-outputs
Author  : GitHub Actions <[email protected]>
Time    : 2023-05-10 08:53:44 +0000
Message : markdown source builds

Auto-generated via {sandpaper}
Source  : 5f57bc7
Branch  : main
Author  : Laurent Gatto <[email protected]>
Time    : 2023-05-10 08:51:20 +0000
Message : Merge pull request #97 from carpentries-incubator/lgmain

use handouts
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