Skip to content

docs: record the November release outcome and two CI lessons - #2346

Merged
robbrad merged 1 commit into
masterfrom
claude/november-notes
Nov 14, 2026
Merged

robbrad merged 1 commit into
masterfrom
claude/november-notes

Conversation

@robbrad

@robbrad robbrad commented Nov 14, 2026 •

Copy link
Copy Markdown
Owner

What

Updates the November section of ISSUE_RESOLUTION_PROGRESS.md now the release is out, and adds two lessons to docs/MAINTAINER_PLAYBOOK.md §12. Docs only: *.md is in bump.yml's paths-ignore, so this does not trigger a release.

  • Progress notes: which version each PR shipped in (0.173.6 / 0.173.7 / 0.173.8), the merge order, what was closed with credit comments, which issues were commented and are left open for the reporter to confirm, the review comments posted on the PRs not taken, and what went differently from plan.
  • Gedling test data is marked resolved (council offices instead of a home address); the note says whether the portal lists that address is unconfirmed.
  • Playbook: lint commit messages with the strict config as well (validate-release-ready.yml keeps body-max-line-length, lint.yml doesn't), and the limits of reading CI logs from the cloud sandbox.

Notes

  • No code changes.
  • Nothing here claims any council works live; where live evidence was unavailable the notes say so.

Test plan

  • Commit message passes commitlint under the strict config (the stricter of the two the repo runs).
  • Docs only; no tests affected.


Generated by Claude Code

Summary by CodeRabbit

  • Documentation
    • Updated November release notes to reflect releases 0.173.6–0.173.8, associated pull requests, and merge and release outcomes.
    • Recorded follow-ups for commit-message checks, fixtures, contributor pull requests, and maintainer decisions, including the completed Gedling test-data replacement.
    • Added guidance on commit-message validation, line-length limits, and accurately reporting CI results when sandbox logs are incomplete.

@coderabbitai

coderabbitai Bot commented Nov 14, 2026 •

Copy link
Copy Markdown
Contributor

Explain Change →

Caution

Review failed

The pull request is closed.

Walkthrough

The release-pass notes now record releases 0.173.6 through 0.173.8 and related follow-ups. The maintainer playbook adds commit-message validation details and guidance on the limits of sandbox CI logs.

Changes: November release pass

Layer / File(s) Summary
Record release outcomes and follow-ups
ISSUE_RESOLUTION_PROGRESS.md
The notes assign releases 0.173.6, 0.173.7, and 0.173.8 to PRs #2343–#2345 and record merge outcomes, follow-ups, and the Gedling test-data replacement.
Update maintainer checks and CI log guidance
docs/MAINTAINER_PLAYBOOK.md
The playbook describes commitlint checks, the 100-character body-line limit, and the limits of sandbox log retrieval. It advises reporting check conclusions accurately and preferring local evidence.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Other

Suggested reviewers: claude

Merge Risk: Minimal · up to 76a3f

This is a documentation-only update to release notes and maintainer guidance, with no change to runtime behavior. No merge-blocking risk remains. Any decision about rewriting repository history for the earlier Gedling fixture is a separate maintainer matter outside this change.

- [ ] 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

@robbrad
robbrad merged commit 5e74ca1 into master Nov 14, 2026
6 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