Skip to content

docs: retire stale roadmap - #275

Merged
David Burg (daviburg) merged 3 commits into
mainfrom
docs/retire-roadmap-issue-251
Sep 25, 2026
Merged

David Burg (daviburg) merged 3 commits into
mainfrom
docs/retire-roadmap-issue-251

Conversation

@daviburg

@daviburg David Burg (daviburg) commented Sep 25, 2026 •

Copy link
Copy Markdown
Member

Summary

Replaces #273 because its fork-based head fails the required PR Source Policy check. The original documentation patch preserves Mia's authorship; a follow-up commit addresses Copilot review.

Validation

  • markdownlint-cli2@0.17.2: 23 Markdown files, zero errors.
  • No tracked ROADMAP.md references remain; the root roadmap is removed.
  • git diff --check: clean.

Closes #251

Replays the approved documentation change from PR #273 on an Azure-owned branch to meet the PR source policy.

Co-authored-by: Dobby <dobby@microsoft.com>
@daviburg
David Burg (daviburg) requested a review from a team as a code owner September 25, 2026 19:29
Copilot AI balanced review requested due to automatic review settings September 25, 2026 19:29

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

Current trigger status and outstanding planning work are not accurately preserved or represented as required by the linked issue.

Get a fresh assessment by requesting another Copilot review.

Review effort: Balanced
Findings: 2 Medium severity · 1 Low severity

Open (3)
What changed in this PR

Retires the stale roadmap and redirects planning toward scoped documentation and GitHub issues.

Changes:

  • Deletes the obsolete roadmap and removes its links.
  • Clarifies README inventory and trigger status guidance.
  • Preserves focused connection and trigger documentation.
File Description
ROADMAP.md Removes stale planning content.
README.md Redirects planning to issues and focused docs.
docs/​triggers.md Updates trigger status and planning link.
docs/​connection-setup.md Removes roadmap reference.
docs/​concepts.md Removes roadmap reference.

💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread docs/triggers.md
Comment thread docs/triggers.md Outdated
Comment thread README.md
@daviburg David Burg (daviburg) self-assigned this Sep 25, 2026
Address Copilot review on PR #275: preserve the action-client decision and link scoped upstream questions.

Co-authored-by: Dobby <dobby@microsoft.com>
Copilot AI review requested due to automatic review settings September 25, 2026 20:40

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

The trigger documentation still shows a ConnectorTrigger constructor signature unsupported by the linked preview extension.

Get a fresh assessment by requesting another Copilot review.

Review effort: Balanced
Findings: 1 Low severity

Open (1)
Resolved since last review (3)

Comment thread docs/triggers.md
Address Copilot review on PR #275 by using the published parameterless trigger binding in the architecture example.

Co-authored-by: Dobby <dobby@microsoft.com>
Copilot AI review requested due to automatic review settings September 25, 2026 20:46

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🟢 Approval recommended

The documentation is consistent, preserves relevant architectural guidance, and satisfies the linked issue’s acceptance criteria.

Review effort: Balanced
Findings: None

Resolved since last review (1)

@daviburg
David Burg (daviburg) merged commit 9f4cbf3 into main Sep 25, 2026
17 checks passed
@daviburg
David Burg (daviburg) deleted the docs/retire-roadmap-issue-251 branch September 25, 2026 21:00
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.

Retire stale ROADMAP.md and move durable architecture to scoped docs

4 participants