Fix broken links in markdown#1539
Conversation
✅ Deploy Preview for moodledevdocs ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify project configuration. |
There was a problem hiding this comment.
Pull request overview
Updates Markdown docs to address broken internal anchors/links across release notes and developer documentation.
Changes:
- Removes or simplifies outdated PHPUnit-upgrade links in older release notes (3.4, 3.11) that pointed at non-existent anchors.
- Fixes/adds explicit heading anchors in docs to support stable internal linking.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| general/releases/3.4.md | Removes a broken link from the PHPUnit upgrade bullet in the 3.4 release notes. |
| general/releases/3.11.md | Removes a broken anchor link reference from the PHPUnit upgrade note in the 3.11 release notes. |
| general/development/policies/codingstyle/index.md | Attempts to fix a heading anchor for the namespaces-within-tests section. |
| docs/guides/testing/index.md | Adds an explicit heading anchor for the Moodle 4.4 / PHPUnit 9.6 upgrade section. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Fixes broken documentation anchors/links in the Moodle docs site by updating headings and removing references to anchors that no longer exist.
Changes:
- Removes outdated/broken deep links to PHPUnit upgrade anchors in older release notes.
- Adds/updates explicit heading anchors intended to stabilize cross-page links.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| general/releases/3.4.md | Removes a broken link/anchor reference in the developer notes. |
| general/releases/3.11.md | Removes a broken anchor link from the developer notes. |
| general/development/policies/codingstyle/index.md | Updates a heading to include an explicit anchor ID for #namespaces-within-tests. |
| docs/guides/testing/index.md | Adds an explicit anchor ID to the “Upgrading unit tests…” heading. |
There was a problem hiding this comment.
Pull request overview
This PR updates the Moodle DevDocs Docusaurus site content to address broken internal links/anchors in Markdown, mainly by adding explicit heading IDs and removing references to anchors that no longer exist.
Changes:
- Removed two broken anchor links from historical release notes (3.4 and 3.11).
- Fixed an invalid heading/anchor declaration in the coding style policy by using an explicit
{#...}ID. - Added an explicit
{#...}heading ID in the PHPUnit testing guide to provide a stable anchor.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| general/releases/3.4.md | Removes a broken link to a non-existent testing-guide anchor in the 3.4 release notes. |
| general/releases/3.11.md | Removes a broken link to a non-existent testing-guide anchor in the 3.11 release notes. |
| general/development/policies/codingstyle/index.md | Fixes the “namespaces-within-tests” section header to use a valid explicit anchor ID. |
| docs/guides/testing/index.md | Adds an explicit anchor ID for the Moodle 4.4 / PHPUnit 9.6 upgrade section. |
| These constants may be then used in your test or plugin code. | ||
|
|
||
| ## Upgrading unit tests to work with Moodle 4.4 and up (PHPUnit 9.6) | ||
| ## Upgrading unit tests to work with Moodle 4.4 and up (PHPUnit 9.6) {#upgrading-unit-tests-to-work-with-moodle-4.4} |
These were just broken anchors.
mickhawkins
left a comment
mickhawkins
left a comment
There was a problem hiding this comment.
Looks good, thanks!
These were just broken anchors.