Skip to content

Add steward skill with PR-handling guidance - #34

Merged
Deep-Sixed merged 1 commit into
mainfrom
claude/loving-lovelace-t15hfr
Sep 30, 2026
Merged

Deep-Sixed merged 1 commit into
mainfrom
claude/loving-lovelace-t15hfr

Conversation

@Deep-Sixed

Copy link
Copy Markdown
Owner

Adds .claude/skills/steward/SKILL.md, the repo-specific guidance agents read before acting on CI and review events for a PR.

It covers:

  • Pre-push checks: the same commands CI runs (ruff check, pyright, python -m pytest tests -q), and a rule not to loosen the ruff pin or the pyright/ruff config to get green.
  • CI: how to read failures from the OS × Python matrix. A failure on one platform or one version is a real bug. Tests and governance checks are never skipped or weakened.
  • Merge conflicts: merge main into the branch (never rebase or force-push), using the existing Merge main (#NN) into <topic> message style. Merged migration numbers are permanent: on a collision, renumber your unmerged migration. When store.py required-object lists conflict, keep both sides.
  • Reviews: address every thread. Any way to bypass a governance rule is blocking and needs a regression test. Bug fixes come with a failing test first.
  • Docs: behavior changes update README.md and docs/agent-guide.md in the same PR.

No code changes. I ran ruff check, pyright and the full suite locally (262 passed) to confirm the commands in the skill work as written.

🤖 Generated with Claude Code

https://claude.ai/code/session_012EkEcEeB6wT1CFEymajDPM


Generated by Claude Code

@Deep-Sixed
Deep-Sixed marked this pull request as ready for review September 30, 2026 07:57
@Deep-Sixed
Deep-Sixed merged commit 6a53970 into main Sep 30, 2026
7 checks passed
Deep-Sixed added a commit that referenced this pull request Sep 30, 2026
# Conflicts:
#	README.md
#	src/quorumgit/gate.py
#	src/quorumgit/git_objects.py
#	tests/test_approval_authority.py
#	tests/test_authorization_binding.py
Deep-Sixed pushed a commit that referenced this pull request Sep 30, 2026
Resolve the README conflict by keeping the consolidated Upgrading
section and folding main's new 010 (protected ref full names) and 011
(quorum consumer separation) notes into it in migration order.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01SHGwY6PLGwr7kQY63qQGq6
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