Skip to content

docs(step2): align tracked closure with reevaluation#32143

Open
TakeshiSawaguchi wants to merge 2 commits into
anomalyco:devfrom
TakeshiSawaguchi:feature/step2-doc-closure-pr-clean
Open

docs(step2): align tracked closure with reevaluation#32143
TakeshiSawaguchi wants to merge 2 commits into
anomalyco:devfrom
TakeshiSawaguchi:feature/step2-doc-closure-pr-clean

Conversation

@TakeshiSawaguchi

@TakeshiSawaguchi TakeshiSawaguchi commented Jun 13, 2026

Copy link
Copy Markdown

Issue for this PR

Closes #32142

Type of change

  • Documentation

What does this PR do?

This PR is a clean, docs-only follow-up to align Step2 reevaluation closure artifacts after the 2026-06-13 verification.

It updates only tracked documentation files:

  • docs/mt5-report-recovery-runbook.md
  • docs/qwen-wag-dell-task-flow.md
  • REVIEW_NOTES.md
  • SPRINT.md

The updates:

  • Keep historical context while adding a canonical 2026-06-13 closure status.
  • Explicitly separate prompts/ (non-tracked operational material) from tracked docs.
  • Consolidate D1/D2/D3 evidence references and close status wording consistently.

This PR does not modify product logic; all operational evidence files and backtest artifacts remain as supporting records for reviewers.

How did you verify your code works?

  • Verified in git diff that only tracked docs are included.
  • Confirmed PR checks check-standards, check-compliance, add-contributor-label, check-duplicates pass.
  • Re-ran gh API checks and confirmed only required docs scope is changed.

Screenshots / recordings

No UI changes.

Checklist

  • I have tested my changes locally
  • I have not included unrelated changes in this PR

@github-actions github-actions Bot added the needs:compliance This means the issue will auto-close after 2 hours. label Jun 13, 2026
@TakeshiSawaguchi

Copy link
Copy Markdown
Author

All checks are passing now:

  • check-standards: pass
  • check-compliance: pass
  • add-contributor-label: pass
  • check-duplicates: pass

PR status indicates mergeStateStatus=BLOCKED, mergeable=MERGEABLE, and latestReviews=0 (no review approvals yet).
This branch is a clean replacement for closed PR #32142 and includes only 4 tracked docs files for Step2 reevaluation closure alignment.
Please approve / merge when branch protection requirements are satisfied.

@TakeshiSawaguchi

Copy link
Copy Markdown
Author

I re-checked blocker details.

Current state: merge is with .

Observed: required check names from ruleset (, , , , ) are not present as runnable check runs for head SHA .

In commit-level check suites, three suites are and contain no check runs (likely fork-approval gating).
runs currently only show:

  • pr-standards: check-standards, check-compliance
  • pr-management: add-contributor-label, check-duplicates

So PR remains blocked by required status mismatch + action-required suites, not by code/merge conflicts.

If possible on maintainer side, please (1) approve-and-run the blocked workflows (or equivalent) and/or (2) confirm required checks policy for doc-only PRs.,

@TakeshiSawaguchi

Copy link
Copy Markdown
Author

I re-checked blocker details.

Current state: merge is blocked with mergeStateStatus=BLOCKED and mergeable=MERGEABLE.

Observed: required check names from ruleset (unit (linux), unit (windows), e2e (linux), e2e (windows), typecheck) are not present as runnable check runs for head SHA a8e7fdc.

In commit-level check suites for this SHA, three suites are conclusion=action_required and contain no check runs (likely fork-approval gating).
Existing completed runs are only pr-standards and pr-management checks:

  • check-standards
  • check-compliance
  • add-contributor-label
  • check-duplicates

So this PR is blocked by required status mismatch + fork action-approval state, not by merge conflicts or code errors. If possible, please approve-and-run the blocked workflows (or equivalent) and/or confirm policy for doc-only PRs.

@github-actions github-actions Bot removed the needs:compliance This means the issue will auto-close after 2 hours. label Jun 13, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Thanks for updating your PR! It now meets our contributing guidelines. 👍

@TakeshiSawaguchi

Copy link
Copy Markdown
Author

Follow-up: I mapped action_required runs to concrete workflow names for #32143.

At head SHA a8e7fdc68...:

  • actions/runs/27456494220 → workflow test (unit (linux) / unit (windows) required)
  • actions/runs/27456494223 → workflow typecheck
  • actions/runs/27456494226 → workflow nix-eval (not in ruleset required list)

test and typecheck are in the required set from ruleset 12269218, and both are currently stopped as action_required (likely maintainer approval needed for self-hosted runner execution on fork).

The required-context issue remains unchanged after PR-body template compliance update.

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