Skip to content

Remove the Settled gh Protocol Entry From TODO.md - #2635

Merged
ptr727 merged 3 commits into
developfrom
feature/auto-2592
Oct 9, 2026
Merged

ptr727 merged 3 commits into
developfrom
feature/auto-2592

Conversation

@ptr727

@ptr727 ptr727 commented Oct 9, 2026

Copy link
Copy Markdown
Owner

Removes the TODO.md entry for the gh git protocol report and its link definition, since #672 is closed and its fix is on main. Also corrects the cluster preamble and the fleet map sentence that counted it.

Closes on promotion: #2592

🤖 Generated with Claude Code

ptr727 and others added 3 commits October 9, 2026 00:44
Issue 672 is closed and its fix is on main, so the entry and its link definition go, per the file's header rule that completed work is deleted once its decision is recorded elsewhere.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…e 672 Entry

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Copilot AI lite review requested due to automatic review settings October 9, 2026 07:46
@coderabbitai

coderabbitai Bot commented Oct 9, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Configuration used: Repository: ptr727/ProjectTemplate/.coderabbit.yaml
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 19725ad3-447f-4d3e-bb39-3f98836bad8f
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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.

🟢 Approval recommended

No unresolved review issues remain.

0 open findings

What changed in this PR

Removes the completed gh protocol TODO item and updates related documentation references.

Changes:

  • Removes the settled #672 entry and link.
  • Updates host-tooling counts and cross-references.
File Description
TODO.md Removes the completed entry and adjusts metadata.
docs/​fleet-map.md Removes the obsolete #672 reference.

🧠 Review effort: Lite


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@codecov

codecov Bot commented Oct 9, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 59.34%. Comparing base (f10164a) to head (dba1195).

Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #2635   +/-   ##
========================================
  Coverage    59.34%   59.34%           
========================================
  Files           16       16           
  Lines         8282     8282           
========================================
  Hits          4915     4915           
  Misses        3367     3367           
Flag Coverage Δ
python-3.13 59.34% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

🟢 Approval recommended

No unresolved review findings remain.

0 open findings

🧠 Review effort: Lite

@ptr727
ptr727 merged commit 6427783 into develop Oct 9, 2026
15 checks passed
ptr727 added a commit that referenced this pull request Oct 9, 2026
Promotes the one change develop holds over main: `TODO.md` drops the
settled entry for the `gh` git protocol report and its link definition,
and the cluster preamble and the `docs/fleet-map.md` sentence that
counted it are corrected to match (#2635).

Fixes #2592

🤖 Generated with [Claude Code](https://claude.com/claude-code)
@ptr727
ptr727 deleted the feature/auto-2592 branch October 9, 2026 14:06
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