Skip to content

validate and fix internal links#125

Merged
zachwaffle4 merged 2 commits into
frcsoftware:mainfrom
samfreund:internal-link-valid
Jul 24, 2026
Merged

validate and fix internal links#125
zachwaffle4 merged 2 commits into
frcsoftware:mainfrom
samfreund:internal-link-valid

Conversation

@samfreund

@samfreund samfreund commented Jul 24, 2026

Copy link
Copy Markdown
Member

Description

This PR adds an internal link checker, which will check links to other pages in the site. This helps ensure that we don't have any broken links with incorrect paths to other pages in the site, as well as ensuring that proper formats are used for links (absolute, forward slash, etc.)

This PR also fixes links that had issues.

supercedes and closes #99

Meta

Merge checklist:

@samfreund
samfreund requested a review from a team as a code owner July 24, 2026 20:01
@github-actions github-actions Bot added Curriculum material Curriculum materials, lessons for students, etc stage1 infra Any infrastructure for building the website or syncing files stage0 labels Jul 24, 2026
@github-actions

Copy link
Copy Markdown

🌐 Preview URL: https://pr-125.frcsoftware.pages.dev

@samfreund

Copy link
Copy Markdown
Member Author

This partially resolves #96, but we still need an external link checker as well. I'll be PRing that shortly.

Signed-off-by: Sam Freund <samf.236@proton.me>
@samfreund
samfreund force-pushed the internal-link-valid branch from 50fdcff to db9349e Compare July 24, 2026 21:54
@zachwaffle4 zachwaffle4 changed the title Validate internal links validate and fix internal links Jul 24, 2026
Signed-off-by: Sam Freund <samf.236@proton.me>
@zachwaffle4
zachwaffle4 merged commit af64f50 into frcsoftware:main Jul 24, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Curriculum material Curriculum materials, lessons for students, etc infra Any infrastructure for building the website or syncing files stage0 stage1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants