Add details to the optional fork-and-PR challenge#181
Add details to the optional fork-and-PR challenge#181alex-ball merged 2 commits intoLibraryCarpentry:mainfrom
Conversation
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:
Rendered Changes🔍 Inspect the changes: https://github.com/LibraryCarpentry/lc-git/compare/md-outputs..md-outputs-PR-181 The following changes were observed in the rendered markdown documents: 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-07-08 09:12:14 +0000 |
|
@srerickson , do these changes look good to you? |
|
Sorry for the delay-- this looks great! |
Auto-generated via `{sandpaper}`
Source : 7f0ab6b
Branch : main
Author : Alex Ball <alex-ball@users.noreply.github.com>
Time : 2025-08-04 08:24:13 +0000
Message : Merge pull request #181 from alex-ball/patch-132
Improve optional fork-and-PR challenge:
1. Fix formatting of heading and introductory paragraph.
2. Explain motivation for this workflow.
3. Replace broken link with illustrated step-by-step instructions.
Auto-generated via `{sandpaper}`
Source : c6aacdc
Branch : md-outputs
Author : GitHub Actions <actions@github.com>
Time : 2025-08-04 08:25:16 +0000
Message : markdown source builds
Auto-generated via `{sandpaper}`
Source : 7f0ab6b
Branch : main
Author : Alex Ball <alex-ball@users.noreply.github.com>
Time : 2025-08-04 08:24:13 +0000
Message : Merge pull request #181 from alex-ball/patch-132
Improve optional fork-and-PR challenge:
1. Fix formatting of heading and introductory paragraph.
2. Explain motivation for this workflow.
3. Replace broken link with illustrated step-by-step instructions.
Auto-generated via `{sandpaper}`
Source : c6aacdc
Branch : md-outputs
Author : GitHub Actions <actions@github.com>
Time : 2025-08-04 08:25:16 +0000
Message : markdown source builds
Auto-generated via `{sandpaper}`
Source : 7f0ab6b
Branch : main
Author : Alex Ball <alex-ball@users.noreply.github.com>
Time : 2025-08-04 08:24:13 +0000
Message : Merge pull request #181 from alex-ball/patch-132
Improve optional fork-and-PR challenge:
1. Fix formatting of heading and introductory paragraph.
2. Explain motivation for this workflow.
3. Replace broken link with illustrated step-by-step instructions.
This pull request addresses several issues with the optional challenge in Episode 5 to clone someone else's repository locally and use it to initiate a pull request:
challengefenced div, not before it in a block quote.The replacement text here fixes the formatting, adds some reasons why this workflow might be preferable, and provides a full set of steps that mirror the previous challenge, including screenshots.
I would welcome any improvements that can be made to these new steps.
Closes #132. Closes #175.