Add subfolder for GitHub Copilot app modernization #48868
+125
−108
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
I worked around it by manually adding the redirect file to the OPS publish config. |
There was a problem hiding this comment.
Pull Request Overview
This PR consolidates GitHub Copilot app modernization content by creating a dedicated subfolder structure and updating all related references. The changes move files from scattered locations into a centralized github-copilot-app-modernization folder within the core porting documentation, improving content organization and maintainability.
- Creates a new subfolder structure for GitHub Copilot app modernization content with its own TOC and index files
- Updates all internal references to point to the new file locations
- Removes the old navigation structure and sets up proper redirections for moved content
Reviewed Changes
Copilot reviewed 22 out of 24 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| docs/navigate/migration-guide/toc.yml | Updates navigation links to point to new subfolder locations |
| docs/navigate/migration-guide/index.yml | Updates landing page links to new file paths |
| docs/navigate/github-copilot-modernize/toc.yml | Removes old navigation file (deleted) |
| docs/core/porting/third-party-deps.md | Updates reference link to new subfolder location |
| docs/core/porting/porting-approaches.md | Updates reference link and fixes minor text improvements |
| docs/core/porting/modernize.md | Updates reference link to new subfolder location |
| docs/core/porting/index.md | Updates reference link to new subfolder location |
| docs/core/porting/includes/github-copilot-suggestion.md | Updates include file references to new locations |
| docs/core/porting/github-copilot-app-modernization/toc.yml | Creates new navigation structure for the subfolder |
| docs/core/porting/github-copilot-app-modernization/overview.md | Updates internal references and include paths |
| docs/core/porting/github-copilot-app-modernization/index.yml | Updates landing page links to relative paths |
| docs/core/porting/github-copilot-app-modernization/how-to-upgrade-with-github-copilot.md | Updates include references and fixes table formatting |
| docs/core/porting/github-copilot-app-modernization/faq.yml | Updates internal references to use relative paths |
| docs/core/porting/framework-overview.md | Updates reference links to new subfolder location |
| docs/core/install/upgrade.md | Updates reference link to new subfolder location |
| docs/azure/toc.yml | Updates Azure navigation links to new file paths |
| docs/azure/migration/appmod/quickstart.md | Updates include reference to new location |
| docs/azure/migration/appmod/deploy.md | Updates include reference to new location |
| docs/azure/migration/appmod/containerization.md | Updates include reference to new location |
| .openpublishing.redirection.navigate.json | Creates redirection for old navigate structure |
| .openpublishing.redirection.core.json | Adds redirections for moved core files |
| .openpublishing.publish.config.json | Registers new redirection file |
Comments suppressed due to low confidence (1)
docs/navigate/migration-guide/toc.yml:1
- The removed troubleshooting entry has malformed query parameters with duplicate
tocandbcparameters separated by?instead of&. This would have caused incorrect URL resolution.
items:
BillWagner
approved these changes
Oct 7, 2025
when you're ready.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Follow up to #48809, which wasn't quite correct.
navigatefolder should only be used as a last resort - it was a necessity since our content was spread out over core/standard/etc. folders so there was no obvious place to add the navigation files).Internal previews
Toggle expand/collapse