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/ImperialCollegeLondon/rse_further_git_course/compare/md-outputs..md-outputs-PR-154 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 2026-01-28 14:17:29 +0000 |
There was a problem hiding this comment.
Pull request overview
This PR focuses on fixing broken internal/download links in the lesson materials and applying a set of small wording/formatting/spelling cleanups across multiple episodes.
Changes:
- Fixes/updates several lesson links (e.g., setup instructions, handout link, CoC links).
- Adds missing downloadable assets under
episodes/files/(handout PDF, recipe zip with history). - Applies consistent formatting/wording tweaks across multiple episodes (e.g., “GitHub” capitalization, punctuation, list formatting).
Reviewed changes
Copilot reviewed 11 out of 13 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| learners/setup.md | Updates the recipe zip download link (currently still mismatched to actual file location). |
| episodes/files/recipe_with_history.zip | Adds recipe repository zip intended to include commit history for exercises. |
| episodes/files/git-course-handout.pdf | Adds course handout PDF referenced from course overview. |
| episodes/10-collaborative_development.md | Wording/capitalization and formatting cleanup. |
| episodes/09-releases_tags.md | Fixes “GitHub” capitalization. |
| episodes/08-continuous_integration.md | Formatting cleanup, terminology tweaks (“PRs”, “macOS”, UI label quoting). |
| episodes/07-managing_contributions.md | Formatting cleanup and minor terminology standardization. |
| episodes/05-pulling_pushing.md | Updates wording to use switch terminology and punctuation cleanup. |
| episodes/04-rewriting_history.md | Fixes wording and updates setup-instructions link target. |
| episodes/02-branching_merging.md | Formatting cleanup and wording clarification around main. |
| episodes/01-collaborating_with_git.md | Formatting cleanup; contains a remaining typo (“need to need to”). |
| episodes/00-course_overview.md | Fixes intro-course reference and updates setup/handout links. |
| CODE_OF_CONDUCT.md | Updates Carpentries CoC/reporting links to new URLs. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Auto-generated via `{sandpaper}`
Source : cba7850
Branch : md-outputs
Author : GitHub Actions <actions@github.com>
Time : 2026-01-28 14:18:44 +0000
Message : markdown source builds
Auto-generated via `{sandpaper}`
Source : e238981
Branch : main
Author : Alex Dewar <alexdewar@users.noreply.github.com>
Time : 2026-01-28 14:17:55 +0000
Message : Merge pull request #154 from ImperialCollegeLondon/fix-links-and-formatting
Various small fixes
The most important thing this PR fixes is a few broken links.
Besides that, I've also tweaked the formatting in a few places as well as changing the wording and fixing spellings etc. (most of these predate the transition to workbench).
I fixed some broken links in the CoC, but I think we probably want to change that file anyway (#146).
I've opened some issues for other small things.
Fixes #133. Fixes #138.
PS -- I've taken the liberty of updating the branch protection rule to include the new status checks.