Skip to content

PLAN Phase 5: point the PR-validation bullet at the #14 tracker's sub-issues - #123

Merged
mmcky merged 1 commit into
mainfrom
plan/phase5-children
Sep 2, 2026
Merged

PLAN Phase 5: point the PR-validation bullet at the #14 tracker's sub-issues#123
mmcky merged 1 commit into
mainfrom
plan/phase5-children

Conversation

@mmcky

@mmcky mmcky commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

One-line edit to PLAN.md Phase 5. The unchecked PR-validation bullet said the three schema decisions "are researched in #14". On 2026-09-02 #14 was conformed to the status-projects tracker contract and the remaining work became its native sub-issues, so the bullet now names them: #119 for the manifest-driven validate() and the PR-validation workflow, and #120, #121, #122 for the column pattern, known_nulls and dtype-vocabulary decisions.

No code or manifest changes.

🤖 Generated with Claude Code

…-issues

The schema decisions and the manifest-driven validate() now have their own
issues (#119 to #122) as native children of the #14 Project tracker, so the
bullet names them rather than saying they are researched in #14.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Copilot AI lite review requested due to automatic review settings September 2, 2026 02:24

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

The change is a documentation-only link/wording update that correctly reflects the current issue-tracking structure and does not affect code or manifests.

Pull request overview

Updates PLAN Phase 5 documentation to reflect that the remaining PR-validation design work is now tracked as sub-issues under #14, rather than being described as “researched in #14”, aligning the plan with the tracker’s current structure.

Changes:

  • Rewords the Phase 5 “PR validation” bullet to explicitly link to the dedicated sub-issues (#119#122) under tracker issue #14.
  • Clarifies which issue covers the shared manifest-driven validate() + PR-validation workflow (#119) versus the three schema decision issues (#120#122).
File summaries
File Description
PLAN.md Updates the Phase 5 PR-validation bullet to reference the specific sub-issues under #14 that now track the remaining work.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@mmcky
mmcky merged commit e318f06 into main Sep 2, 2026
2 checks passed
@mmcky
mmcky deleted the plan/phase5-children branch September 2, 2026 02:42
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