Skip to content

Document Postponed Homelab Runner Design - #890

Merged
ptr727 merged 3 commits into
developfrom
feature/889-runner-selection
Aug 21, 2026
Merged

Document Postponed Homelab Runner Design#890
ptr727 merged 3 commits into
developfrom
feature/889-runner-selection

Conversation

@ptr727

@ptr727 ptr727 commented Aug 21, 2026

Copy link
Copy Markdown
Owner

Summary

  • Preserve the proposed homelab runner-selection and security design as a trackable artifact.
  • Keep Copilot code review on GitHub-hosted infrastructure unless ARC is adopted.
  • Record that implementation is postponed because the expected Actions savings do not justify the operating complexity.

Verification

  • Full Python lint, type, test, spec, distribution, repository, and prose gates passed.
  • python3 scripts/docker_lint.py completed all six linters successfully.
  • The commit carries a verified signature.

Tracking

References #889. This pull request documents a postponed design and does not close the issue.

Copilot AI lite review requested due to automatic review settings August 21, 2026 03:57

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This pull request adds a postponed design document for issue #889, preserving the proposed homelab runner-selection and security boundary as a trackable artifact while explicitly keeping Copilot code review on GitHub-hosted infrastructure.

Changes:

  • Add a new plan document describing the proposed runner-selection contract, trust checks, and canary rollout steps.
  • Record that implementation is postponed due to ROI and operational complexity.

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

Comment thread docs/homelab-runner-selection-plan.md
Copilot AI review requested due to automatic review settings August 21, 2026 04:06

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

Suppressed comments (1)

docs/homelab-runner-selection-plan.md:9

  • The Table of Contents block should follow the repo convention: add a ## Table of Contents <!-- omit from toc --> heading and let the Markdown All in One extension maintain the list (see .agents/skills/comment-and-doc-style/SKILL.md and existing docs like docs/token-efficiency-plan.md). The current bullet list appears as a hand-authored TOC without the standard heading.
- [Status](#status)
- [Decision Outcome](#decision-outcome)
- [Objective](#objective)
- [Non-Goals](#non-goals)
- [Primary Constraint: Copilot Code Review](#primary-constraint-copilot-code-review)

Copilot AI review requested due to automatic review settings August 21, 2026 04:11
@ptr727

ptr727 commented Aug 21, 2026

Copy link
Copy Markdown
Owner Author

Suppressed comments (1)

docs/homelab-runner-selection-plan.md:9 The Table of Contents block should follow the repo convention by adding the standard heading.

Fixed in 01b8206. Added ## Table of Contents <!-- omit from toc --> above the generated list. The list remains extension-generated and unchanged.

Review round: #890 (review)

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

@ptr727
ptr727 marked this pull request as ready for review August 21, 2026 04:16
@ptr727
ptr727 merged commit 0f2edcd into develop Aug 21, 2026
8 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