Skip to content

Document test execution expectations in project README - #287

Merged
ProtocolWarden merged 2 commits into
mainfrom
goal/1b13bad9
Jun 14, 2026
Merged

Document test execution expectations in project README#287
ProtocolWarden merged 2 commits into
mainfrom
goal/1b13bad9

Conversation

@ProtocolWarden

Copy link
Copy Markdown
Owner

Auto-generated by Operations Center execution.

Goal

Document test execution expectations in project README

Definition of done (complete ALL before finishing)

  1. Complete the task in its ENTIRETY — every acceptance criterion and every
    file the task implies (implementation, tests, and docs as applicable). Do
    not leave TODOs, stubs, or 'follow-up' gaps; a partial change is rejected
    in review.
  2. Add or update tests/checks that prove the work is correct.
  3. Run the repository's test suite and linters/formatters and make them
    pass locally. If anything fails, fix it before finishing — do not hand
    off a red build.
  4. Only consider the task done when the full change is in place AND verified
    green. The PR you open should be mergeable as-is.

@ProtocolWarden

ProtocolWarden commented Jun 14, 2026

Copy link
Copy Markdown
Owner Author

Resolved: new push — automated review resumed

Needs human attention (reason=ci_persistently_red). Left open — not merged (unresolved) and not closed (work preserved).

CI has not gone green after 20 checks (1 failing: audit: failure). Not merged (red CI) and not closed (work preserved) — needs a human to fix CI.

…n task.md

T8: tests/unit/test_documentation_accuracy.py validates README/config/CI content
via filesystem reads and subprocess calls — no src imports needed by design.
R2: task.md was missing ## Overall Plan and ## Current Stage required sections.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@ProtocolWarden

ProtocolWarden commented Jun 14, 2026

Copy link
Copy Markdown
Owner Author

Resolved: CI green on unchanged head — test suite validates implementation; automated review resumed

Needs human attention (reason=ci_never_settled). Left open — not merged (unresolved) and not closed (work preserved).

CI has not settled-green on the current head after 21 checks (6 still running: Type check (ty), Snapshot validation, audit, Test (pytest), Custodian doctor). Not merged (CI incomplete) and not closed (work preserved) — needs a human to investigate stuck CI.

@ProtocolWarden
ProtocolWarden merged commit 4a5a24b into main Jun 14, 2026
17 checks passed
@ProtocolWarden
ProtocolWarden deleted the goal/1b13bad9 branch June 14, 2026 07:52
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.

1 participant