8000 fix: correct broken links in cookbook documentation by GeekTrainer · Pull Request #93 · github/copilot-sdk · GitHub 8000
[go: up one dir, main page]

Skip to content

Conversation

@GeekTrainer
Copy link
Contributor

Summary

This PR fixes broken and incorrect links found in the cookbook documentation.

Changes

  1. Fixed duplicate folder paths - All recipe links in cookbook/README.md incorrectly included '/cookbook/' twice in the path (e.g., dotnet/cookbook/error-handling.mddotnet/error-handling.md)
  2. Fixed CONTRIBUTING.md path - Updated to use correct relative path ../CONTRIBUTING.md
  3. Fixed typo - Renamed pr-visialisation.mdpr-visualization.md in nodejs cookbook and updated references

Testing

Verified all links now resolve to existing files in the repository.

Impact

  • Fixes navigation from cookbook README to recipe pages
  • Fixes navigation to CONTRIBUTING.md
  • Corrects filename spelling for Node.js PR visualization recipe

- Renamed pr-visialisation.md to pr-visualization.md
- Updated link reference in cookbook/nodejs/README.md
- Fixed all recipe links in cookbook/README.md that incorrectly included '/cookbook/' twice in the path
- Updated dotnet, nodejs, python, and go recipe links to use correct relative paths
- Also fixed CONTRIBUTING.md link to use correct relative path (../CONTRIBUTING.md)
@GeekTrainer GeekTrainer requested a review from a team as a code owner January 22, 2026 17:20
Copilot AI review requested due to automatic review settings January 22, 2026 17:20
Copy link
Contributor
Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes broken documentation links in the cookbook section by correcting duplicate folder paths, fixing a relative path to CONTRIBUTING.md, and correcting a filename typo.

Changes:

  • Removed duplicate /cookbook/ folder in recipe links across all language sections (.NET, Node.js, Python, Go)
  • Updated CONTRIBUTING.md reference to use correct relative path (../CONTRIBUTING.md)
  • Fixed typo in Node.js PR visualization filename from pr-visialisation.md to pr-visualization.md

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.

File Description
cookbook/README.md Corrected all recipe links by removing duplicate /cookbook/ folder paths and fixed CONTRIBUTING.md relative path
cookbook/nodejs/README.md Fixed spelling in PR visualization link from pr-visialisation.md to pr-visualization.md

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Contributor
@SteveSandersonMS SteveSandersonMS left a comment

Choose a reason for hiding this comment

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

Thanks @GeekTrainer!

@SteveSandersonMS SteveSandersonMS added this pull request to the merge queue Jan 22, 2026
Merged via the queue into github:main with commit 2c3ceb4 Jan 22, 2026
0 of 12 checks passed
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.

2 participants

0