Skip to content

Conversation

@abigailsparling
Copy link
Contributor

added missing link to course resources

added missing link to course resources
@github-actions
Copy link

github-actions bot commented Nov 13, 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/LibraryCarpentry/lc-git/compare/md-outputs..md-outputs-PR-191

The following changes were observed in the rendered markdown documents:

 04-review.md | 2 +-
 md5sum.txt   | 2 +-
 2 files changed, 2 insertions(+), 2 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-11-17 16:00:48 +0000

github-actions bot pushed a commit that referenced this pull request Nov 13, 2025
@alex-ball
Copy link
Contributor

Thanks for spotting and fixing this, @abigailsparling .

Delving back into the history of this lesson, it looks like this has been a problem for as long as lessons have been presented as one episode per page, since there has never been a list of links below that phrase. I'm grateful to you for calling it out.

In order to keep the terminology consistent, would you mind changing "course" to "lesson"?

@alex-ball alex-ball added the status:waiting for response Waiting for Contributor to respond to maintainers' comments or update PR label Nov 17, 2025
@alex-ball alex-ball self-assigned this Nov 17, 2025
@abigailsparling
Copy link
Contributor Author

@alex-ball Change made!

github-actions bot pushed a commit that referenced this pull request Nov 17, 2025
@alex-ball
Copy link
Contributor

@abigailsparling , thank you!

@alex-ball alex-ball merged commit d3df864 into LibraryCarpentry:main Nov 17, 2025
3 checks passed
github-actions bot pushed a commit that referenced this pull request Nov 17, 2025
Auto-generated via `{sandpaper}`
Source  : d3df864
Branch  : main
Author  : Alex Ball <alex-ball@users.noreply.github.com>
Time    : 2025-11-17 16:32:25 +0000
Message : Merge pull request #191 from abigailsparling/abigailsparling-patch-1

Replace "links below" (of which there aren't any) in 04-review.md with link to links
github-actions bot pushed a commit that referenced this pull request Nov 17, 2025
Auto-generated via `{sandpaper}`
Source  : 7adbc94
Branch  : md-outputs
Author  : GitHub Actions <actions@github.com>
Time    : 2025-11-17 16:33:37 +0000
Message : markdown source builds

Auto-generated via `{sandpaper}`
Source  : d3df864
Branch  : main
Author  : Alex Ball <alex-ball@users.noreply.github.com>
Time    : 2025-11-17 16:32:25 +0000
Message : Merge pull request #191 from abigailsparling/abigailsparling-patch-1

Replace "links below" (of which there aren't any) in 04-review.md with link to links
github-actions bot pushed a commit that referenced this pull request Nov 18, 2025
Auto-generated via `{sandpaper}`
Source  : 7adbc94
Branch  : md-outputs
Author  : GitHub Actions <actions@github.com>
Time    : 2025-11-17 16:33:37 +0000
Message : markdown source builds

Auto-generated via `{sandpaper}`
Source  : d3df864
Branch  : main
Author  : Alex Ball <alex-ball@users.noreply.github.com>
Time    : 2025-11-17 16:32:25 +0000
Message : Merge pull request #191 from abigailsparling/abigailsparling-patch-1

Replace "links below" (of which there aren't any) in 04-review.md with link to links
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status:waiting for response Waiting for Contributor to respond to maintainers' comments or update PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants