Skip to content

docs: fix stale GitHub docs links - #15

Open
JSap0914 wants to merge 1 commit into
phac-modelling-hub:mainfrom
JSap0914:fix/github-flow-docs-link-5
Open

docs: fix stale GitHub docs links#15
JSap0914 wants to merge 1 commit into
phac-modelling-hub:mainfrom
JSap0914:fix/github-flow-docs-link-5

Conversation

@JSap0914

@JSap0914 JSap0914 commented Jul 5, 2026

Copy link
Copy Markdown

Summary

  • update stale GitHub Docs URLs in projects/version-control.qmd
  • correct the same-page GitHub Flow anchor casing to match the Quarto heading ID

Fixes #5.

Evidence

Issue #5 reports that the GitHub Flow link in the Git workflow section is dead. Direct checks showed the old GitHub Flow URL returns HTTP 404 and the current GitHub Docs URL returns HTTP 200.

While checking the same page, two other GitHub Docs URLs also returned HTTP 404; their current official targets return HTTP 200.

Duplicate checks

Before editing and again before PR creation, open PR searches for the GitHub Flow URL, the stale GitHub Docs paths, and this head branch returned no matches.

Verification

  • git diff --check
  • direct curl -I -L checks for each old/new external URL pair
  • npm_config_script_shell=/bin/bash npx --yes markdown-link-check projects/version-control.qmd

The markdown link check now passes the updated external links. It still reports the Quarto custom same-page anchors #making-changes, #github-flow, and #git-basics as 404 because the checker does not resolve {#...} heading IDs; those IDs are defined in this file.

Copilot AI review requested due to automatic review settings July 5, 2026 08:13

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

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.

Dead link for Github Flow

2 participants