Skip to content

Split the Remaining Over-Length Sentences in the Host Setup Docs - #3598

Merged
ptr727 merged 1 commit into
developfrom
feature/3531-md-host-setup-residue
Oct 29, 2026
Merged

ptr727 merged 1 commit into
developfrom
feature/3531-md-host-setup-residue

Conversation

@ptr727

@ptr727 ptr727 commented Oct 29, 2026

Copy link
Copy Markdown
Owner

Part of #3531

The five sentences are orchestrator rulings applied verbatim, not author rewordings.

🤖 Generated with Claude Code

Part of #3531

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Copilot AI lite review requested due to automatic review settings October 29, 2026 04:48
@ptr727

ptr727 commented Oct 29, 2026

Copy link
Copy Markdown
Owner Author

A recorded local strict-review pass covers head 6478d6a4f4074d2bc16a7b226b394676456231fe, the content this pull request carries at that commit against develop, and it recorded 0 findings.

@coderabbitai

coderabbitai Bot commented Oct 29, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

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

Run configuration
  • 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.

🟡 Changes recommended

Two unresolved nit-level findings remain: reworded sentences in README.md and docs/host-setup.md depart from the verbatim wording the PR description cites, and a split sentence in docs/host-setup.md drops its reason and exceeds the 25-word cap.

2 open issues
What changed in this PR

This PR splits over-length sentences in the agent-enablement docs so they fit the sentence-length cap. The source text is hub-verbatim rulings, so the split should keep the original wording as far as possible.

Changes:

  • Splits the long "What This Repo Is" paragraph and the human-contribution paragraph in the root README, turning the list of maintainer duties into bullets.
  • Splits the WSL and Docker Desktop paragraph in the Windows host-setup README into shorter sentences.
  • Splits the "host needs no Python" and "host-tools.json" paragraphs in the host-setup doc.
File Description
README.md Splits the purpose paragraph and turns the human-contribution duties into a bullet list. Some wording changed during the split.
host-setup/​windows/​README.md Splits the install-tools.ps1 WSL and Docker Desktop paragraph into shorter sentences with minor rewording.
docs/​host-setup.md Splits the Python-free and host-tools paragraphs. The Python-free split still needs a further cut to meet the cap.

Review effort: Lite

Comment thread README.md Resolved
Comment thread docs/host-setup.md Resolved
@codecov

codecov Bot commented Oct 29, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (develop@ef727b4). Learn more about missing BASE report.

Additional details and impacted files
@@            Coverage Diff             @@
##             develop    #3598   +/-   ##
==========================================
  Coverage           ?   81.60%           
==========================================
  Files              ?       26           
  Lines              ?     9070           
  Branches           ?        0           
==========================================
  Hits               ?     7402           
  Misses             ?     1668           
  Partials           ?        0           
Flag Coverage Δ
python-3.13 81.60% <ø> (?)

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.

@ptr727
ptr727 merged commit c59f7db into develop Oct 29, 2026
14 checks passed
@ptr727
ptr727 deleted the feature/3531-md-host-setup-residue branch October 29, 2026 04:59
ptr727 added a commit that referenced this pull request Oct 29, 2026
…down (#3600)

## Summary

Promotes `develop` to `main`, carrying the sentence-length splits of the
hub-only Markdown:

- #3594: splits them in scripts/README.md.
- #3593: splits them in STANDUP, AUDIT, RESYNC, and OPERATIONS.
- #3591: splits them in TODO.md.
- #3596: splits them in the agent-safety Markdown.
- #3595: splits them in the host setup docs.
- #3592: splits them in reusable-workflows and fleet-map.
- #3598: splits them in the remaining host setup sentences.
- #3597: splits them in the spec and repo-config docs.
- #3599: splits them in the docs, catalog READMEs, and HISTORY.

This is part of #3531, which stays open.

🤖 Generated with [Claude Code](https://claude.com/claude-code)
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