Skip to content

fix(docs): Fix broken ecosystem link in roadmap - #2375

Merged
Michael Flanakin (flanakin) merged 2 commits into
devfrom
flanakin/fix-roadmap-ecosystem-link
Oct 28, 2026
Merged

Michael Flanakin (flanakin) merged 2 commits into
devfrom
flanakin/fix-roadmap-ecosystem-link

Conversation

@flanakin

Copy link
Copy Markdown
Collaborator

🤖 [AI] Line 39 of roadmap.md linked to finops-toolkit-ecosystem.md, which doesn't exist (added in #2359). This broke Pester DocsLinks on every PR into dev. The link now points to the ecosystem page that the overview already uses. DocsLinks tests pass locally (1563/0).

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Verified the fix. finops-toolkit-ecosystem.md doesn't exist, and the new URL matches what finops-toolkit-overview.md and help-options.md already use, as well as the same line on flanakin/v15-prep. Merging this unblocks the DocsLinks Pester failure on every PR into dev. Note: /finops-toolkit/ecosystem currently returns 404 on the live site because docs/ecosystem.md hasn't been published yet; it'll resolve with the next site deploy.

Comment thread docs-mslearn/toolkit/roadmap.md Resolved
@flanakin
Michael Flanakin (flanakin) merged commit c40ce1d into dev Oct 28, 2026
5 of 6 checks passed
@flanakin
Michael Flanakin (flanakin) deleted the flanakin/fix-roadmap-ecosystem-link branch October 28, 2026 15:40

This branch was previously deployed

1 inactive deployment
ftk-pr — dad26b70 Deployed Oct 28, 2026 by flanakin via Clean up PR resources #86
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs: Review 👀 PR that is ready to be reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants