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

Some issue 526 fixes #546

Merged
merged 4 commits into from
Feb 18, 2025
Merged

Conversation

RicCampbell
Copy link
Contributor

@RicCampbell RicCampbell commented Feb 17, 2025

Addresses some issue #526 points

Links to cheatsheet changed to rstudio github page as opposed to direct pdf downloads (dplyr, tidyr, ggplot2, RMarkdown link in resources list was already a github link).

Have left tidyverse link in episode 4 as did not appear as link in edit screen, no links added to episodes 5-7.

One link removed from RMarkdown resource list 'The Ecosystem of R Markdown' (no obvious replacement found), all other links are working so not altered.

Link to dplyr cheatsheet changed to rstudio repo github link.
Link to tidyr cheatsheet changed to rstudio repo github link.
Link to ggplot cheatsheet changed to rstudio repo github link x2
Removal of The Ecosystem of R Markdown link as no longer exists.
Copy link

github-actions bot commented Feb 17, 2025

Thank you!

Thank you for your pull request 😃

🤖 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:

  • 🎯 correct output
  • 🖼️ correct figures
  • ❓ new warnings
  • ‼️ new errors

Rendered Changes

🔍 Inspect the changes: https://github.com/datacarpentry/r-socialsci/compare/md-outputs..md-outputs-PR-546

The following changes were observed in the rendered markdown documents:

 03-dplyr.md                                        |   2 +-
 04-tidyr.md                                        |  22 ++++++++++-----------
 05-ggplot2.md                                      |   4 ++--
 06-rmarkdown.md                                    |   3 +--
 fig/05-ggplot2-rendered-adding-colors-1.png        | Bin 18618 -> 18814 bytes
 fig/05-ggplot2-rendered-adding-jitter-1.png        | Bin 9189 -> 9187 bytes
 fig/05-ggplot2-rendered-adding-width-height-1.png  | Bin 9264 -> 9223 bytes
 fig/05-ggplot2-rendered-boxplot-exercise-1.png     | Bin 10325 -> 10354 bytes
 ...-ggplot2-rendered-boxplot-exercise-factor-1.png | Bin 21148 -> 21283 bytes
 fig/05-ggplot2-rendered-boxplot-with-jitter-1.png  | Bin 17384 -> 17410 bytes
 fig/05-ggplot2-rendered-color-by-species-1.png     | Bin 20902 -> 20249 bytes
 fig/05-ggplot2-rendered-scatter-challenge-1.png    | Bin 19766 -> 19799 bytes
 fig/05-ggplot2-rendered-violin-plot-1.png          | Bin 28058 -> 28202 bytes
 md5sum.txt                                         |   8 ++++----
 14 files changed, 19 insertions(+), 20 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.

⏱️ Updated at 2025-02-18 13:14:33 +0000

github-actions bot pushed a commit that referenced this pull request Feb 18, 2025
@juanfung
Copy link
Contributor

@RicCampbell very nice, thank you!

@juanfung juanfung merged commit f369e11 into datacarpentry:main Feb 18, 2025
3 checks passed
github-actions bot pushed a commit that referenced this pull request Feb 18, 2025
Auto-generated via `{sandpaper}`
Source  : f369e11
Branch  : main
Author  : Juan Fung <[email protected]>
Time    : 2025-02-18 15:04:14 +0000
Message : Merge pull request #546 from RicCampbell/issue-526-fixes

Some issue 526 fixes
github-actions bot pushed a commit that referenced this pull request Feb 18, 2025
Auto-generated via `{sandpaper}`
Source  : d042ecf
Branch  : md-outputs
Author  : GitHub Actions <[email protected]>
Time    : 2025-02-18 15:05:44 +0000
Message : markdown source builds

Auto-generated via `{sandpaper}`
Source  : f369e11
Branch  : main
Author  : Juan Fung <[email protected]>
Time    : 2025-02-18 15:04:14 +0000
Message : Merge pull request #546 from RicCampbell/issue-526-fixes

Some issue 526 fixes
@jessesadler
Copy link
Contributor

Great changes! Thanks so much @RicCampbell.

@RicCampbell RicCampbell deleted the issue-526-fixes branch February 18, 2025 16:54
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.

3 participants