Skip to content

Conversation

Copy link

Copilot AI commented Nov 15, 2025

Fix Broken Links in SONiC Repository

Progress Checklist

  • Explore repository structure and identify all markdown and HTML files
  • Install and configure link checking tool (markdown-link-check)
  • Scan repository for broken links
  • Categorize broken links by type
  • Fix malformed anchor link in TACACS documentation
  • Fix URLs with spaces in release notes
  • Add missing LICENSE file
  • Verify all fixes with link checker
  • Run code review
  • Run security scan

Fixed Issues

1. Added Missing LICENSE File

  • Created Apache 2.0 LICENSE file referenced in README.md
  • The README states the project is licensed under Apache 2.0, so added the standard Apache 2.0 license text

2. Fixed Malformed Anchor Link

  • File: doc/tacacs-passkey/TACACSPLUS_PASSKEY_ENCRYPTION.md
  • Issue: Line 10 had @show-cli-changes instead of #show-cli-changes
  • Fix: Changed @ to # to create proper markdown anchor link

3. Fixed URLs with Spaces

Fixed markdown links with space after opening parenthesis in 4 release notes files:

  • doc/release-notes/SONiC_201911_Release_Notes.md
  • doc/release-notes/SONiC_202006_Release_Notes.md
  • doc/release-notes/SONiC_202012_Release_Notes.md
  • doc/release-notes/SONiC_202106_Release_Notes.md

Changed [here]( https://...) to [here](https://...) to fix the broken link syntax.

Verification

All fixes have been verified:

  • ✓ LICENSE file now exists and README.md link works
  • ✓ TACACS anchor link is correct
  • ✓ All release notes URLs are properly formatted
Original prompt

Check this repo for any broken links


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

@linux-foundation-easycla
Copy link

CLA Not Signed

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

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.

3 participants